~alpine/aports

This thread contains a patchset. You're looking at the original emails, but you may wish to use the patch review UI. Review patch
7 3

[PATCH] community/py3-tinycss2: upgrade to 1.1.1

Details
Message ID
<20211123072649.3636-1-sir@cmpwn.com>
DKIM signature
missing
Download raw message
Patch: +14 -8
---
 community/py3-tinycss2/APKBUILD | 22 ++++++++++++++--------
 1 file changed, 14 insertions(+), 8 deletions(-)

diff --git a/community/py3-tinycss2/APKBUILD b/community/py3-tinycss2/APKBUILD
index 0aa76a8e4d..1b6580d53a 100644
--- a/community/py3-tinycss2/APKBUILD
+++ b/community/py3-tinycss2/APKBUILD
@@ -2,31 +2,37 @@
# Maintainer: Drew DeVault <sir@cmpwn.com>
pkgname=py3-tinycss2
_pyname=tinycss2
pkgver=1.1.0
pkgrel=1
pkgver=1.1.1
pkgrel=0
pkgdesc="Low-level CSS parser for Python"
url="https://pypi.python.org/pypi/tinycss2"
arch="noarch"
license="BSD-3-Clause"
depends="py3-webencodings"
checkdepends="py3-pytest py3-pytest-flake8 py3-pytest-isort py3-pytest-cov"
makedepends="python3-dev py3-setuptools"
makedepends="python3-dev py3-setuptools pyproject2setuppy"
_pypiprefix="${_pyname%${_pyname#?}}"
source="https://files.pythonhosted.org/packages/source/$_pypiprefix/$_pyname/$_pyname-$pkgver.tar.gz"
builddir="$srcdir"/$_pyname-$pkgver
replaces="py-tinycss py2-tinycss py3-tinycss"
options="!check" # upstream tests are broken

prepare() {
	# Run your damn test suite BEFORE you ship your package
	sed -i pyproject.toml -e 's/--isort //'
}

build() {
	python3 setup.py build
	python3 -m pyproject2setuppy.main build
}

check() {
	python3 setup.py pytest
	pytest
}

package() {
	python3 setup.py install --prefix=/usr --root="$pkgdir"
	python3 -m pyproject2setuppy.main install --prefix=/usr --root="$pkgdir"
}

sha512sums="218d28511802afc89a608def03084f5d460512a41d47280b5c8bfe72314c8ab3b14c98fbb0200bea6b8509c4d6f0107410d340e2d7f1830aaa92669a2699211e  tinycss2-1.1.0.tar.gz"
sha512sums="
297ac51cf4eb9a063e88d677b8cf2511ff6c459c7a385ee8ba6bb4b9d9b0d8482f5caa1cc6511d3edefbe8e8ee284d72fedc154a5372880cd7d9a9a26aab1a62  tinycss2-1.1.1.tar.gz
"
-- 
2.34.0
Details
Message ID
<164052384086.14324.5310477798080804045.gitlab.27693.328ff68011109126fa8a5c8a96226e59b96e536f@listserv.local>
In-Reply-To
<20211123072649.3636-1-sir@cmpwn.com> (view parent)
DKIM signature
missing
Download raw message
Tests fail with:

  tests/test_tinycss2.py:7: in <module>
      from tinycss2 import (
  E   ModuleNotFoundError: No module named 'tinycss2'

You probably need to add `export PYTHONPATH=$builddir` to `check()`.

-- 
via https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/27693#note_201252
Details
Message ID
<CGPGDZZ4V2VC.1JWKX19278QNB@taiga>
In-Reply-To
<164052384086.14324.5310477798080804045.gitlab.27693.328ff68011109126fa8a5c8a96226e59b96e536f@listserv.local> (view parent)
DKIM signature
missing
Download raw message
This is one of multiple issues with the tests, which is why I have them
disabled. The other, more important issue, is that the test data is
distributed separately from the rest of the upstream source code.
Details
Message ID
<164054422652.14324.6850831721889115433.gitlab.27693.328ff68011109126fa8a5c8a96226e59b96e536f@listserv.local>
In-Reply-To
<164052384086.14324.5310477798080804045.gitlab.27693.328ff68011109126fa8a5c8a96226e59b96e536f@listserv.local> (view parent)
DKIM signature
missing
Download raw message
Drew DeVault <sir@cmpwn.com> replied via email:

```
This is one of multiple issues with the tests, which is why I have them
disabled. The other, more important issue, is that the test data is
distributed separately from the rest of the upstream source code.

```

-- 
via https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/27693#note_201312
Details
Message ID
<164054759232.14324.4567415095405048744.gitlab.27693.328ff68011109126fa8a5c8a96226e59b96e536f@listserv.local>
In-Reply-To
<164054422652.14324.6850831721889115433.gitlab.27693.328ff68011109126fa8a5c8a96226e59b96e536f@listserv.local> (view parent)
DKIM signature
missing
Download raw message
This patch removes the `options="!check"` line, so now the tests fail.

-- 
via https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/27693#note_201320
Details
Message ID
<CGPHNKL7DW6I.1MZNGD69RUD87@taiga>
In-Reply-To
<164054759232.14324.4567415095405048744.gitlab.27693.328ff68011109126fa8a5c8a96226e59b96e536f@listserv.local> (view parent)
DKIM signature
missing
Download raw message
Hm, right. I was looking at this package on a different computer I had
prepared the patch from. Will prep a v2.
Details
Message ID
<164054779488.14324.8476452177048148191.gitlab.27693.328ff68011109126fa8a5c8a96226e59b96e536f@listserv.local>
In-Reply-To
<164054759232.14324.4567415095405048744.gitlab.27693.328ff68011109126fa8a5c8a96226e59b96e536f@listserv.local> (view parent)
DKIM signature
missing
Download raw message
Drew DeVault <sir@cmpwn.com> replied via email:

```
Hm, right. I was looking at this package on a different computer I had
prepared the patch from. Will prep a v2.

```

-- 
via https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/27693#note_201322
Details
Message ID
<164054856502.14324.9847140795934462068.gitlab.27693.65ca3ba909df63d076d96dfce0ac1c8b4b1c8f81@listserv.local>
In-Reply-To
<20211123072649.3636-1-sir@cmpwn.com> (view parent)
DKIM signature
missing
Download raw message
Superseded by !28897

-- 
via https://gitlab.alpinelinux.org/alpine/aports/-/merge_requests/27693#note_201328
Reply to thread Export thread (mbox)