Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] python3-zstandard: update to 0.22.0
@ 2024-03-31 21:21 r-ricci
  2024-03-31 21:40 ` tornaria
  2024-03-31 21:42 ` [PR PATCH] [Closed]: " ahesford
  0 siblings, 2 replies; 4+ messages in thread
From: r-ricci @ 2024-03-31 21:21 UTC (permalink / raw)
  To: ml

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

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

https://github.com/r-ricci/void-packages python3-zstandard
https://github.com/void-linux/void-packages/pull/49636

python3-zstandard: update to 0.22.0
This happens when using system's zstd:

```
$ python3 -c 'import zstandard'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/lib/python3.12/site-packages/zstandard/__init__.py", line 39, in <module>
    from .backend_c import *  # type: ignore
    ^^^^^^^^^^^^^^^^^^^^^^^^
ImportError: zstd C API versions mismatch; Python bindings were not compiled/linked against expected zstd version (10506 returned by the lib, 10506 hardcoded in zstd headers, 10505 hardcoded in the cext)
```

<!-- 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 [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-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
-->


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

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

From 692062999655a8711504e265ff8b49cc1e12ed01 Mon Sep 17 00:00:00 2001
From: Roberto Ricci <io@r-ricci.it>
Date: Sun, 31 Mar 2024 19:28:28 +0200
Subject: [PATCH] python3-zstandard: update to 0.22.0

- use bundled zstd
- remove unnecessary pre_check()
---
 srcpkgs/python3-zstandard/template | 21 ++++++++-------------
 1 file changed, 8 insertions(+), 13 deletions(-)

diff --git a/srcpkgs/python3-zstandard/template b/srcpkgs/python3-zstandard/template
index d59cbb09b965fc..76edc16fe6f7ec 100644
--- a/srcpkgs/python3-zstandard/template
+++ b/srcpkgs/python3-zstandard/template
@@ -1,11 +1,14 @@
 # Template file for 'python3-zstandard'
 pkgname=python3-zstandard
-version=0.21.0
-revision=3
+version=0.22.0
+revision=1
 build_style=python3-module
-make_build_args="--system-zstd"
+# If the system's zstd is used and its version is not the same as
+# the bundled zstd, importing the python module fails.
+# https://github.com/indygreg/python-zstandard/issues/48#issuecomment-753344624
+#make_build_args="--system-zstd"
 hostmakedepends="python3-setuptools"
-makedepends="python3-devel libzstd-devel"
+makedepends="python3-devel"
 depends="python3"
 checkdepends="python3-hypothesis"
 short_desc="Python bindings to the Zstandard (zstd) compression library"
@@ -14,15 +17,7 @@ license="BSD-3-Clause"
 homepage="https://github.com/indygreg/python-zstandard"
 changelog="https://raw.githubusercontent.com/indygreg/python-zstandard/main/docs/news.rst"
 distfiles="https://github.com/indygreg/python-zstandard/archive/${version}.tar.gz"
-checksum=15adc6bfa629d48b0bb658e9eae94c484adb66a28738fa780478eebeb41599a5
-
-pre_check() {
-	# The skipped test ignored due to failure, see:
-	# https://github.com/indygreg/python-zstandard/issues/147#issuecomment-874278901
-	if [ "${XBPS_TARGET_MACHINE%-musl}" = i686 ]; then
-		vsed -i -e '/def test_estimated_compression_context_size/s/test//' tests/test_data_structures*.py
-	fi
-}
+checksum=34ce7ef020afca1344c538a778e2a2e30dc43b11509aa4cb5cf076228d959ca7
 
 post_install() {
 	vlicense LICENSE

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

* Re: python3-zstandard: update to 0.22.0
  2024-03-31 21:21 [PR PATCH] python3-zstandard: update to 0.22.0 r-ricci
@ 2024-03-31 21:40 ` tornaria
  2024-03-31 21:42 ` [PR PATCH] [Closed]: " ahesford
  1 sibling, 0 replies; 4+ messages in thread
From: tornaria @ 2024-03-31 21:40 UTC (permalink / raw)
  To: ml

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

New comment by tornaria on void-packages repository

https://github.com/void-linux/void-packages/pull/49636#issuecomment-2028911613

Comment:
Dupe of #49606. 

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

* Re: [PR PATCH] [Closed]: python3-zstandard: update to 0.22.0
  2024-03-31 21:21 [PR PATCH] python3-zstandard: update to 0.22.0 r-ricci
  2024-03-31 21:40 ` tornaria
@ 2024-03-31 21:42 ` ahesford
  1 sibling, 0 replies; 4+ messages in thread
From: ahesford @ 2024-03-31 21:42 UTC (permalink / raw)
  To: ml

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

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

python3-zstandard: update to 0.22.0
https://github.com/void-linux/void-packages/pull/49636

Description:
This happens when using system's zstd:

```
$ python3 -c 'import zstandard'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
  File "/usr/lib/python3.12/site-packages/zstandard/__init__.py", line 39, in <module>
    from .backend_c import *  # type: ignore
    ^^^^^^^^^^^^^^^^^^^^^^^^
ImportError: zstd C API versions mismatch; Python bindings were not compiled/linked against expected zstd version (10506 returned by the lib, 10506 hardcoded in zstd headers, 10505 hardcoded in the cext)
```

<!-- 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 [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-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
-->


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

* [PR PATCH] python3-zstandard: update to 0.22.0.
@ 2024-03-30 13:20 unspecd
  0 siblings, 0 replies; 4+ messages in thread
From: unspecd @ 2024-03-30 13:20 UTC (permalink / raw)
  To: ml

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

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

https://github.com/unspecd/void-packages python3-zstandard
https://github.com/void-linux/void-packages/pull/49606

python3-zstandard: update to 0.22.0.
Also: use bundled libzstd instead of the system one: python3-zstandard requires the specific version of libzstd, 1.5.5.

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

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

<!--
#### New package
- This new package conforms to the [package requirements](https://github.com/void-linux/void-packages/blob/master/CONTRIBUTING.md#package-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, (x86_64-musl)
- I built this PR locally for these architectures (if supported. mark crossbuilds):


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

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

From ae46e563b9ea906d92f9dc73ce87ab529e135cba Mon Sep 17 00:00:00 2001
From: Evgeny Ermakov <evgeny.v.ermakov@gmail.com>
Date: Sat, 30 Mar 2024 12:10:42 +1100
Subject: [PATCH] python3-zstandard: update to 0.22.0.

Also: use bundled libzstd instead of the system one:
python3-zstandard requires the specific version of libzstd, 1.5.5.
---
 srcpkgs/python3-zstandard/template | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/srcpkgs/python3-zstandard/template b/srcpkgs/python3-zstandard/template
index d59cbb09b965fc..ff4a8754920d1f 100644
--- a/srcpkgs/python3-zstandard/template
+++ b/srcpkgs/python3-zstandard/template
@@ -1,11 +1,10 @@
 # Template file for 'python3-zstandard'
 pkgname=python3-zstandard
-version=0.21.0
-revision=3
+version=0.22.0
+revision=1
 build_style=python3-module
-make_build_args="--system-zstd"
 hostmakedepends="python3-setuptools"
-makedepends="python3-devel libzstd-devel"
+makedepends="python3-devel"
 depends="python3"
 checkdepends="python3-hypothesis"
 short_desc="Python bindings to the Zstandard (zstd) compression library"
@@ -14,7 +13,7 @@ license="BSD-3-Clause"
 homepage="https://github.com/indygreg/python-zstandard"
 changelog="https://raw.githubusercontent.com/indygreg/python-zstandard/main/docs/news.rst"
 distfiles="https://github.com/indygreg/python-zstandard/archive/${version}.tar.gz"
-checksum=15adc6bfa629d48b0bb658e9eae94c484adb66a28738fa780478eebeb41599a5
+checksum=34ce7ef020afca1344c538a778e2a2e30dc43b11509aa4cb5cf076228d959ca7
 
 pre_check() {
 	# The skipped test ignored due to failure, see:

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

end of thread, other threads:[~2024-03-31 21:42 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2024-03-31 21:21 [PR PATCH] python3-zstandard: update to 0.22.0 r-ricci
2024-03-31 21:40 ` tornaria
2024-03-31 21:42 ` [PR PATCH] [Closed]: " ahesford
  -- strict thread matches above, loose matches on Subject: below --
2024-03-30 13:20 [PR PATCH] " unspecd

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).