* Re: New package: python3-pytest-flake8-1.0.7
2021-02-18 15:19 [PR PATCH] New package: python3-pytest-flake8-1.0.7 paper42
@ 2021-02-18 15:26 ` FollieHiyuki
2021-02-18 15:37 ` [PR PATCH] [Updated] " paper42
` (8 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: FollieHiyuki @ 2021-02-18 15:26 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 217 bytes --]
New comment by FollieHiyuki on void-packages repository
https://github.com/void-linux/void-packages/pull/28842#issuecomment-781422448
Comment:
Why not use the default `do_check()` from build style? It doesn't work?
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PR PATCH] [Updated] New package: python3-pytest-flake8-1.0.7
2021-02-18 15:19 [PR PATCH] New package: python3-pytest-flake8-1.0.7 paper42
2021-02-18 15:26 ` FollieHiyuki
@ 2021-02-18 15:37 ` paper42
2021-02-18 15:41 ` paper42
` (7 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: paper42 @ 2021-02-18 15:37 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1650 bytes --]
There is an updated pull request by paper42 against master on the void-packages repository
https://github.com/paper42/void-packages 0001-New-package-python3-pytest-flake8-1.0.7.patch
https://github.com/void-linux/void-packages/pull/28842
New package: python3-pytest-flake8-1.0.7
<!-- Mark items with [x] where applicable -->
#### General
- [x] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!--
#### Does it build and run successfully?
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
- [ ] aarch64-musl
- [ ] armv7l
- [ ] armv6l-musl
-->
A patch file from https://github.com/void-linux/void-packages/pull/28842.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-0001-New-package-python3-pytest-flake8-1.0.7.patch-28842.patch --]
[-- Type: text/x-diff, Size: 1346 bytes --]
From 902981970cad35c8e8215e96c961eb7cc2512555 Mon Sep 17 00:00:00 2001
From: Paper <paper@tilde.institute>
Date: Thu, 18 Feb 2021 16:15:04 +0100
Subject: [PATCH] New package: python3-pytest-flake8-1.0.7
---
srcpkgs/python3-pytest-flake8/template | 24 ++++++++++++++++++++++++
1 file changed, 24 insertions(+)
create mode 100644 srcpkgs/python3-pytest-flake8/template
diff --git a/srcpkgs/python3-pytest-flake8/template b/srcpkgs/python3-pytest-flake8/template
new file mode 100644
index 00000000000..b2368da1b74
--- /dev/null
+++ b/srcpkgs/python3-pytest-flake8/template
@@ -0,0 +1,24 @@
+# Template file for 'python3-pytest-flake8'
+pkgname=python3-pytest-flake8
+version=1.0.7
+revision=1
+wrksrc=pytest-flake8-${version}
+build_style=python3-module
+hostmakedepends="python3-setuptools"
+depends="python3-pytest flake8"
+checkdepends="$depends"
+short_desc="Pytest plugin to check FLAKE8 requirements"
+maintainer="Paper <paper@tilde.institute>"
+license="BSD-3-Clause"
+homepage="https://pypi.org/project/pytest-flake8/"
+changelog="https://raw.githubusercontent.com/tholo/pytest-flake8/master/CHANGELOG"
+distfiles="${PYPI_SITE}/p/pytest-flake8/pytest-flake8-${version}.tar.gz"
+checksum=f0259761a903563f33d6f099914afef339c085085e643bee8343eb323b32dd6b
+
+do_check() {
+ PYTHONPATH=. pytest
+}
+
+post_install() {
+ vlicense LICENSE
+}
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: New package: python3-pytest-flake8-1.0.7
2021-02-18 15:19 [PR PATCH] New package: python3-pytest-flake8-1.0.7 paper42
2021-02-18 15:26 ` FollieHiyuki
2021-02-18 15:37 ` [PR PATCH] [Updated] " paper42
@ 2021-02-18 15:41 ` paper42
2021-02-18 18:59 ` [PR REVIEW] " ahesford
` (6 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: paper42 @ 2021-02-18 15:41 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 218 bytes --]
New comment by paper42 on void-packages repository
https://github.com/void-linux/void-packages/pull/28842#issuecomment-781433372
Comment:
@FollieHiyuki you were right, I forgot. Now I added PYTHONPATH, so I need it.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PR REVIEW] New package: python3-pytest-flake8-1.0.7
2021-02-18 15:19 [PR PATCH] New package: python3-pytest-flake8-1.0.7 paper42
` (2 preceding siblings ...)
2021-02-18 15:41 ` paper42
@ 2021-02-18 18:59 ` ahesford
2021-02-19 14:50 ` paper42
` (5 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: ahesford @ 2021-02-18 18:59 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 307 bytes --]
New review comment by ahesford on void-packages repository
https://github.com/void-linux/void-packages/pull/28842#discussion_r578670581
Comment:
You don't need this. The build style will automatically call `python3 -m pytest` when `python3-pytest` is available, which implicitly adds `.` to `PYTHONPATH`.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: New package: python3-pytest-flake8-1.0.7
2021-02-18 15:19 [PR PATCH] New package: python3-pytest-flake8-1.0.7 paper42
` (3 preceding siblings ...)
2021-02-18 18:59 ` [PR REVIEW] " ahesford
@ 2021-02-19 14:50 ` paper42
2021-02-20 15:37 ` [PR PATCH] [Updated] " paper42
` (4 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: paper42 @ 2021-02-19 14:50 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 491 bytes --]
New comment by paper42 on void-packages repository
https://github.com/void-linux/void-packages/pull/28842#issuecomment-782122348
Comment:
> If this package isn't being pulled into `checkdepends` for other end-user packages, does it provide value on its own?
no, but the tests depending on this package are patched out from at least python3-numpy-stl. Then, at least python3-pep517, python3-cairocffi and radicale have their tests disabled **partially** because of this missing package.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PR PATCH] [Updated] New package: python3-pytest-flake8-1.0.7
2021-02-18 15:19 [PR PATCH] New package: python3-pytest-flake8-1.0.7 paper42
` (4 preceding siblings ...)
2021-02-19 14:50 ` paper42
@ 2021-02-20 15:37 ` paper42
2021-02-22 14:41 ` ahesford
` (3 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: paper42 @ 2021-02-20 15:37 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1650 bytes --]
There is an updated pull request by paper42 against master on the void-packages repository
https://github.com/paper42/void-packages 0001-New-package-python3-pytest-flake8-1.0.7.patch
https://github.com/void-linux/void-packages/pull/28842
New package: python3-pytest-flake8-1.0.7
<!-- Mark items with [x] where applicable -->
#### General
- [x] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!--
#### Does it build and run successfully?
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
- [ ] aarch64-musl
- [ ] armv7l
- [ ] armv6l-musl
-->
A patch file from https://github.com/void-linux/void-packages/pull/28842.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-0001-New-package-python3-pytest-flake8-1.0.7.patch-28842.patch --]
[-- Type: text/x-diff, Size: 1301 bytes --]
From 8c7314e6e722901131fabda18890f1063d20e0e9 Mon Sep 17 00:00:00 2001
From: Paper <paper@tilde.institute>
Date: Thu, 18 Feb 2021 16:15:04 +0100
Subject: [PATCH] New package: python3-pytest-flake8-1.0.7
---
srcpkgs/python3-pytest-flake8/template | 20 ++++++++++++++++++++
1 file changed, 20 insertions(+)
create mode 100644 srcpkgs/python3-pytest-flake8/template
diff --git a/srcpkgs/python3-pytest-flake8/template b/srcpkgs/python3-pytest-flake8/template
new file mode 100644
index 00000000000..701d19fad33
--- /dev/null
+++ b/srcpkgs/python3-pytest-flake8/template
@@ -0,0 +1,20 @@
+# Template file for 'python3-pytest-flake8'
+pkgname=python3-pytest-flake8
+version=1.0.7
+revision=1
+wrksrc=pytest-flake8-${version}
+build_style=python3-module
+hostmakedepends="python3-setuptools"
+depends="python3-pytest flake8"
+checkdepends="$depends"
+short_desc="Pytest plugin to check FLAKE8 requirements"
+maintainer="Paper <paper@tilde.institute>"
+license="BSD-3-Clause"
+homepage="https://pypi.org/project/pytest-flake8/"
+changelog="https://raw.githubusercontent.com/tholo/pytest-flake8/master/CHANGELOG"
+distfiles="${PYPI_SITE}/p/pytest-flake8/pytest-flake8-${version}.tar.gz"
+checksum=f0259761a903563f33d6f099914afef339c085085e643bee8343eb323b32dd6b
+
+post_install() {
+ vlicense LICENSE
+}
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: New package: python3-pytest-flake8-1.0.7
2021-02-18 15:19 [PR PATCH] New package: python3-pytest-flake8-1.0.7 paper42
` (5 preceding siblings ...)
2021-02-20 15:37 ` [PR PATCH] [Updated] " paper42
@ 2021-02-22 14:41 ` ahesford
2021-03-02 19:53 ` [PR PATCH] [Updated] " paper42
` (2 subsequent siblings)
9 siblings, 0 replies; 11+ messages in thread
From: ahesford @ 2021-02-22 14:41 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 306 bytes --]
New comment by ahesford on void-packages repository
https://github.com/void-linux/void-packages/pull/28842#issuecomment-783422202
Comment:
If adding this package is sufficient to make tests in `python3-numpy-stl` work without the `pre_check` patch, please drop that patch in this PR. No need to revbump.
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PR PATCH] [Updated] New package: python3-pytest-flake8-1.0.7
2021-02-18 15:19 [PR PATCH] New package: python3-pytest-flake8-1.0.7 paper42
` (6 preceding siblings ...)
2021-02-22 14:41 ` ahesford
@ 2021-03-02 19:53 ` paper42
2021-03-02 19:54 ` paper42
2021-03-12 13:47 ` [PR PATCH] [Merged]: " ahesford
9 siblings, 0 replies; 11+ messages in thread
From: paper42 @ 2021-03-02 19:53 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1650 bytes --]
There is an updated pull request by paper42 against master on the void-packages repository
https://github.com/paper42/void-packages 0001-New-package-python3-pytest-flake8-1.0.7.patch
https://github.com/void-linux/void-packages/pull/28842
New package: python3-pytest-flake8-1.0.7
<!-- Mark items with [x] where applicable -->
#### General
- [x] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!--
#### Does it build and run successfully?
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
- [ ] aarch64-musl
- [ ] armv7l
- [ ] armv6l-musl
-->
A patch file from https://github.com/void-linux/void-packages/pull/28842.patch is attached
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-0001-New-package-python3-pytest-flake8-1.0.7.patch-28842.patch --]
[-- Type: text/x-diff, Size: 2478 bytes --]
From 150c0d95023db3cc210a6760bf404ec91be53c95 Mon Sep 17 00:00:00 2001
From: Paper <paper@tilde.institute>
Date: Thu, 18 Feb 2021 16:15:04 +0100
Subject: [PATCH 1/2] New package: python3-pytest-flake8-1.0.7
---
srcpkgs/python3-pytest-flake8/template | 20 ++++++++++++++++++++
1 file changed, 20 insertions(+)
create mode 100644 srcpkgs/python3-pytest-flake8/template
diff --git a/srcpkgs/python3-pytest-flake8/template b/srcpkgs/python3-pytest-flake8/template
new file mode 100644
index 00000000000..701d19fad33
--- /dev/null
+++ b/srcpkgs/python3-pytest-flake8/template
@@ -0,0 +1,20 @@
+# Template file for 'python3-pytest-flake8'
+pkgname=python3-pytest-flake8
+version=1.0.7
+revision=1
+wrksrc=pytest-flake8-${version}
+build_style=python3-module
+hostmakedepends="python3-setuptools"
+depends="python3-pytest flake8"
+checkdepends="$depends"
+short_desc="Pytest plugin to check FLAKE8 requirements"
+maintainer="Paper <paper@tilde.institute>"
+license="BSD-3-Clause"
+homepage="https://pypi.org/project/pytest-flake8/"
+changelog="https://raw.githubusercontent.com/tholo/pytest-flake8/master/CHANGELOG"
+distfiles="${PYPI_SITE}/p/pytest-flake8/pytest-flake8-${version}.tar.gz"
+checksum=f0259761a903563f33d6f099914afef339c085085e643bee8343eb323b32dd6b
+
+post_install() {
+ vlicense LICENSE
+}
From 1d3d810520df78d8682e808507bd48bca9d4bf34 Mon Sep 17 00:00:00 2001
From: Paper <paper@tilde.institute>
Date: Tue, 2 Mar 2021 20:53:15 +0100
Subject: [PATCH 2/2] python3-numpy-stl: add python3-pytest-flake8 dep
---
srcpkgs/python3-numpy-stl/template | 6 +-----
1 file changed, 1 insertion(+), 5 deletions(-)
diff --git a/srcpkgs/python3-numpy-stl/template b/srcpkgs/python3-numpy-stl/template
index 83756860cda..42fa5e4ad93 100644
--- a/srcpkgs/python3-numpy-stl/template
+++ b/srcpkgs/python3-numpy-stl/template
@@ -8,7 +8,7 @@ build_helper="numpy"
hostmakedepends="python3-setuptools python3-Cython"
makedepends="python3-devel"
depends="python3-setuptools python3-numpy python3-utils"
-checkdepends="$depends python3-pytest-cov python3-docutils"
+checkdepends="$depends python3-pytest-cov python3-pytest-flake8 python3-docutils"
short_desc="Python3 library to make working with STL files fast and easy"
maintainer="Karl Nilsson <karl.robert.nilsson@gmail.com>"
license="BSD-3-Clause"
@@ -22,10 +22,6 @@ pre_build() {
sed -i '/setup_requires=/d' setup.py
}
-pre_check() {
- vsed -i pytest.ini -e s/--flake8//
-}
-
pkg_install() {
vlicense LICENSE
}
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: New package: python3-pytest-flake8-1.0.7
2021-02-18 15:19 [PR PATCH] New package: python3-pytest-flake8-1.0.7 paper42
` (7 preceding siblings ...)
2021-03-02 19:53 ` [PR PATCH] [Updated] " paper42
@ 2021-03-02 19:54 ` paper42
2021-03-12 13:47 ` [PR PATCH] [Merged]: " ahesford
9 siblings, 0 replies; 11+ messages in thread
From: paper42 @ 2021-03-02 19:54 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 325 bytes --]
New comment by paper42 on void-packages repository
https://github.com/void-linux/void-packages/pull/28842#issuecomment-789172250
Comment:
> If adding this package is sufficient to make tests in `python3-numpy-stl` work without the `pre_check` patch, please drop that patch in this PR. No need to revbump.
@ahesford done
^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: [PR PATCH] [Merged]: New package: python3-pytest-flake8-1.0.7
2021-02-18 15:19 [PR PATCH] New package: python3-pytest-flake8-1.0.7 paper42
` (8 preceding siblings ...)
2021-03-02 19:54 ` paper42
@ 2021-03-12 13:47 ` ahesford
9 siblings, 0 replies; 11+ messages in thread
From: ahesford @ 2021-03-12 13:47 UTC (permalink / raw)
To: ml
[-- Attachment #1: Type: text/plain, Size: 1451 bytes --]
There's a merged pull request on the void-packages repository
New package: python3-pytest-flake8-1.0.7
https://github.com/void-linux/void-packages/pull/28842
Description:
<!-- Mark items with [x] where applicable -->
#### General
- [x] This is a new package and it conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements)
#### Have the results of the proposed changes been tested?
- [ ] I use the packages affected by the proposed changes on a regular basis and confirm this PR works for me
- [ ] I generally don't use the affected packages but briefly tested this PR
<!--
If GitHub CI cannot be used to validate the build result (for example, if the
build is likely to take several hours), make sure to
[skip CI](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration).
When skipping CI, uncomment and fill out the following section.
Note: for builds that are likely to complete in less than 2 hours, it is not
acceptable to skip CI.
-->
<!--
#### Does it build and run successfully?
(Please choose at least one native build and, if supported, at least one cross build. More are better.)
- [ ] I built this PR locally for my native architecture, (ARCH-LIBC)
- [ ] I built this PR locally for these architectures (if supported. mark crossbuilds):
- [ ] aarch64-musl
- [ ] armv7l
- [ ] armv6l-musl
-->
^ permalink raw reply [flat|nested] 11+ messages in thread