Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] python3-pikepdf: update to 5.4.0.
@ 2022-07-17 21:07 oynqr
  2022-07-20  1:29 ` classabbyamp
                   ` (4 more replies)
  0 siblings, 5 replies; 6+ messages in thread
From: oynqr @ 2022-07-17 21:07 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1534 bytes --]

There is a new pull request by oynqr against master on the void-packages repository

https://github.com/oynqr/void-packages python3-pikepdf
https://github.com/void-linux/void-packages/pull/38115

python3-pikepdf: update to 5.4.0.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- 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
-->

Plain setuptools_scm is now used, so there's no need to patch out setuptools_scm_git_archive anymore. Since 5.1.4 the egg-info directory needs to be on PYTHONPATH to run tests, so I copy it to the correct location as a workaround. Also added new required and optional depends/checkdepends.

A patch file from https://github.com/void-linux/void-packages/pull/38115.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-python3-pikepdf-38115.patch --]
[-- Type: text/x-diff, Size: 2704 bytes --]

From a89fca40969f42cab88a9f2a06a3a33ca9f8b018 Mon Sep 17 00:00:00 2001
From: Philipp David <pd@3b.pm>
Date: Sun, 12 Jun 2022 22:22:22 +0200
Subject: [PATCH] python3-pikepdf: update to 5.4.0.

---
 .../remove_pytest-xdist_dependency.patch      | 19 -------------------
 srcpkgs/python3-pikepdf/template              | 15 ++++++++-------
 2 files changed, 8 insertions(+), 26 deletions(-)
 delete mode 100644 srcpkgs/python3-pikepdf/patches/remove_pytest-xdist_dependency.patch

diff --git a/srcpkgs/python3-pikepdf/patches/remove_pytest-xdist_dependency.patch b/srcpkgs/python3-pikepdf/patches/remove_pytest-xdist_dependency.patch
deleted file mode 100644
index ea844d5e4d7c..000000000000
--- a/srcpkgs/python3-pikepdf/patches/remove_pytest-xdist_dependency.patch
+++ /dev/null
@@ -1,19 +0,0 @@
---- a/pyproject.toml
-+++ b/pyproject.toml
-@@ -35,7 +35,7 @@
- '''
- 
- [tool.cibuildwheel]
--test-command = "pytest -nauto {project}/tests"
-+test-command = "pytest {project}/tests"
- test-extras = "test"
- skip = "cp310-*"
- test-skip = "*_arm64 *_universal2:arm64"
-@@ -82,7 +82,6 @@
- minversion = "6.0"
- norecursedirs = ["lib", ".pc", ".git", "venv"]
- testpaths = ["tests"]
--addopts = "-n auto"
- 
- [tool.coverage.run]
- concurrency = ["multiprocessing"]
diff --git a/srcpkgs/python3-pikepdf/template b/srcpkgs/python3-pikepdf/template
index b742def96c97..406647b801a0 100644
--- a/srcpkgs/python3-pikepdf/template
+++ b/srcpkgs/python3-pikepdf/template
@@ -1,21 +1,22 @@
 # Template file for 'python3-pikepdf'
 pkgname=python3-pikepdf
-version=5.1.3
+version=5.4.0
 revision=1
 wrksrc="pikepdf-${version}"
 build_style=python3-module
 hostmakedepends="python3-pybind11 python3-setuptools_scm python3-wheel"
 makedepends="libqpdf-devel python3-pybind11"
-depends="python3-lxml python3-packaging python3-Pillow python3-setuptools"
-checkdepends="python3-dateutil python3-hypothesis python3-psutil python3-pytest
- $depends"
+depends="jbig2dec python3-deprecation python3-lxml python3-packaging
+ python3-Pillow python3-setuptools"
+checkdepends="poppler python3-dateutil python3-hypothesis python3-psutil
+ python3-pytest python3-pytest-timeout python3-pytest-xdist $depends"
 short_desc="Python library for reading and writing PDF files"
 maintainer="Philipp David <pd@3b.pm>"
 license="MPL-2.0"
 homepage="https://github.com/pikepdf/pikepdf"
 distfiles="${PYPI_SITE}/p/pikepdf/pikepdf-${version}.tar.gz"
-checksum=c34e4239661d2ddf23caa1c4256f636c866ce8069a5052a2bf8ee06e3cae22f3
+checksum=d397a061959a5cd7d9869a381dda005eccfef59c33dd3eba6a8e19ce036168be
 
-pre_build() {
-	vsed -e '/setuptools_scm_git_archive/d' -i setup.py
+pre_check() {
+	cp -r src/pikepdf.egg-info "$(cd build/lib* && pwd)"
 }

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: python3-pikepdf: update to 5.4.0.
  2022-07-17 21:07 [PR PATCH] python3-pikepdf: update to 5.4.0 oynqr
@ 2022-07-20  1:29 ` classabbyamp
  2022-07-20 10:20 ` oynqr
                   ` (3 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: classabbyamp @ 2022-07-20  1:29 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 441 bytes --]

New comment by classabbyamp on void-packages repository

https://github.com/void-linux/void-packages/pull/38115#issuecomment-1189705311

Comment:
we should still remove the `-n auto`s. when `python3-xdist` is added to `checkdepends`, it uses `XBPS_MAKEJOBS` to set the number of jobs when running pytest.
https://github.com/void-linux/void-packages/blob/ef0f2bd2d00c9de1144e661f52eaa926fc27a8b2/common/build-style/python3-module.sh#L28-L34

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: python3-pikepdf: update to 5.4.0.
  2022-07-17 21:07 [PR PATCH] python3-pikepdf: update to 5.4.0 oynqr
  2022-07-20  1:29 ` classabbyamp
@ 2022-07-20 10:20 ` oynqr
  2022-07-20 10:27 ` oynqr
                   ` (2 subsequent siblings)
  4 siblings, 0 replies; 6+ messages in thread
From: oynqr @ 2022-07-20 10:20 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 217 bytes --]

New comment by oynqr on void-packages repository

https://github.com/void-linux/void-packages/pull/38115#issuecomment-1190097412

Comment:
I just tried it and `XBPS_MAKEJOBS` is still respected during the test phase.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: python3-pikepdf: update to 5.4.0.
  2022-07-17 21:07 [PR PATCH] python3-pikepdf: update to 5.4.0 oynqr
  2022-07-20  1:29 ` classabbyamp
  2022-07-20 10:20 ` oynqr
@ 2022-07-20 10:27 ` oynqr
  2022-07-20 10:30 ` [PR PATCH] [Updated] " oynqr
  2022-07-20 16:43 ` [PR PATCH] [Merged]: " classabbyamp
  4 siblings, 0 replies; 6+ messages in thread
From: oynqr @ 2022-07-20 10:27 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 256 bytes --]

New comment by oynqr on void-packages repository

https://github.com/void-linux/void-packages/pull/38115#issuecomment-1190097412

Comment:
I just tried it and `XBPS_MAKEJOBS` is still respected during the test phase.
I'll still add back the patch though.

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PR PATCH] [Updated] python3-pikepdf: update to 5.4.0.
  2022-07-17 21:07 [PR PATCH] python3-pikepdf: update to 5.4.0 oynqr
                   ` (2 preceding siblings ...)
  2022-07-20 10:27 ` oynqr
@ 2022-07-20 10:30 ` oynqr
  2022-07-20 16:43 ` [PR PATCH] [Merged]: " classabbyamp
  4 siblings, 0 replies; 6+ messages in thread
From: oynqr @ 2022-07-20 10:30 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1539 bytes --]

There is an updated pull request by oynqr against master on the void-packages repository

https://github.com/oynqr/void-packages python3-pikepdf
https://github.com/void-linux/void-packages/pull/38115

python3-pikepdf: update to 5.4.0.
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- 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
-->

Plain setuptools_scm is now used, so there's no need to patch out setuptools_scm_git_archive anymore. Since 5.1.4 the egg-info directory needs to be on PYTHONPATH to run tests, so I copy it to the correct location as a workaround. Also added new required and optional depends/checkdepends.

A patch file from https://github.com/void-linux/void-packages/pull/38115.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-python3-pikepdf-38115.patch --]
[-- Type: text/x-diff, Size: 1745 bytes --]

From bca6b17ff35ee59fd4952e7cd7d01ae5fd91e87e Mon Sep 17 00:00:00 2001
From: Philipp David <pd@3b.pm>
Date: Wed, 20 Jul 2022 12:28:17 +0200
Subject: [PATCH] python3-pikepdf: update to 5.4.0.

---
 srcpkgs/python3-pikepdf/template | 15 ++++++++-------
 1 file changed, 8 insertions(+), 7 deletions(-)

diff --git a/srcpkgs/python3-pikepdf/template b/srcpkgs/python3-pikepdf/template
index b742def96c97..406647b801a0 100644
--- a/srcpkgs/python3-pikepdf/template
+++ b/srcpkgs/python3-pikepdf/template
@@ -1,21 +1,22 @@
 # Template file for 'python3-pikepdf'
 pkgname=python3-pikepdf
-version=5.1.3
+version=5.4.0
 revision=1
 wrksrc="pikepdf-${version}"
 build_style=python3-module
 hostmakedepends="python3-pybind11 python3-setuptools_scm python3-wheel"
 makedepends="libqpdf-devel python3-pybind11"
-depends="python3-lxml python3-packaging python3-Pillow python3-setuptools"
-checkdepends="python3-dateutil python3-hypothesis python3-psutil python3-pytest
- $depends"
+depends="jbig2dec python3-deprecation python3-lxml python3-packaging
+ python3-Pillow python3-setuptools"
+checkdepends="poppler python3-dateutil python3-hypothesis python3-psutil
+ python3-pytest python3-pytest-timeout python3-pytest-xdist $depends"
 short_desc="Python library for reading and writing PDF files"
 maintainer="Philipp David <pd@3b.pm>"
 license="MPL-2.0"
 homepage="https://github.com/pikepdf/pikepdf"
 distfiles="${PYPI_SITE}/p/pikepdf/pikepdf-${version}.tar.gz"
-checksum=c34e4239661d2ddf23caa1c4256f636c866ce8069a5052a2bf8ee06e3cae22f3
+checksum=d397a061959a5cd7d9869a381dda005eccfef59c33dd3eba6a8e19ce036168be
 
-pre_build() {
-	vsed -e '/setuptools_scm_git_archive/d' -i setup.py
+pre_check() {
+	cp -r src/pikepdf.egg-info "$(cd build/lib* && pwd)"
 }

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PR PATCH] [Merged]: python3-pikepdf: update to 5.4.0.
  2022-07-17 21:07 [PR PATCH] python3-pikepdf: update to 5.4.0 oynqr
                   ` (3 preceding siblings ...)
  2022-07-20 10:30 ` [PR PATCH] [Updated] " oynqr
@ 2022-07-20 16:43 ` classabbyamp
  4 siblings, 0 replies; 6+ messages in thread
From: classabbyamp @ 2022-07-20 16:43 UTC (permalink / raw)
  To: ml

[-- Attachment #1: Type: text/plain, Size: 1379 bytes --]

There's a merged pull request on the void-packages repository

python3-pikepdf: update to 5.4.0.
https://github.com/void-linux/void-packages/pull/38115

Description:
<!-- Uncomment relevant sections and delete options which are not applicable -->

#### Testing the changes
- I tested the changes in this PR: **YES**

<!--
#### New package
- This new package conforms to the [quality requirements](https://github.com/void-linux/void-packages/blob/master/Manual.md#quality-requirements): **YES**|**NO**
-->

<!-- Note: If the build is likely to take more than 2 hours, please add ci skip tag as described in
https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#continuous-integration
and test at least one native build and, if supported, at least one cross build.
Ignore this section if this PR is not skipping CI.
-->
<!--
#### Local build testing
- 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
-->

Plain setuptools_scm is now used, so there's no need to patch out setuptools_scm_git_archive anymore. Since 5.1.4 the egg-info directory needs to be on PYTHONPATH to run tests, so I copy it to the correct location as a workaround. Also added new required and optional depends/checkdepends.

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2022-07-20 16:43 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-07-17 21:07 [PR PATCH] python3-pikepdf: update to 5.4.0 oynqr
2022-07-20  1:29 ` classabbyamp
2022-07-20 10:20 ` oynqr
2022-07-20 10:27 ` oynqr
2022-07-20 10:30 ` [PR PATCH] [Updated] " oynqr
2022-07-20 16:43 ` [PR PATCH] [Merged]: " classabbyamp

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).