Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] python-cffi: split python3-cffi to a new package
@ 2023-06-16 21:34 paper42
  2023-06-17 17:04 ` [PR PATCH] [Merged]: " paper42
  0 siblings, 1 reply; 2+ messages in thread
From: paper42 @ 2023-06-16 21:34 UTC (permalink / raw)
  To: ml

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

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

https://github.com/paper42/void-packages py2split-cffi
https://github.com/void-linux/void-packages/pull/44483

python-cffi: split python3-cffi to a new package
<!-- Uncomment relevant sections and delete options which are not applicable -->

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

@ahesford 
<!--
#### 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/44483.patch is attached

[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: github-pr-py2split-cffi-44483.patch --]
[-- Type: text/x-diff, Size: 3677 bytes --]

From b63aef37956a238859792b1d64f1ba0c914de55b Mon Sep 17 00:00:00 2001
From: Michal Vasilek <michal@vasilek.cz>
Date: Fri, 16 Jun 2023 22:23:41 +0200
Subject: [PATCH] python-cffi: split python3-cffi to a new package

---
 srcpkgs/python-cffi/template  | 29 +++--------------------------
 srcpkgs/python3-cffi          |  1 -
 srcpkgs/python3-cffi/template | 33 +++++++++++++++++++++++++++++++++
 3 files changed, 36 insertions(+), 27 deletions(-)
 delete mode 120000 srcpkgs/python3-cffi
 create mode 100644 srcpkgs/python3-cffi/template

diff --git a/srcpkgs/python-cffi/template b/srcpkgs/python-cffi/template
index 2d0c4c50832f..911adc8f8c85 100644
--- a/srcpkgs/python-cffi/template
+++ b/srcpkgs/python-cffi/template
@@ -2,11 +2,10 @@
 pkgname=python-cffi
 version=1.15.1
 revision=2
-build_style=python-module
-hostmakedepends="python-setuptools python3-setuptools libffi-devel"
-makedepends="python-devel python3-devel libffi-devel"
+build_style=python2-module
+hostmakedepends="python-setuptools libffi-devel"
+makedepends="python-devel libffi-devel"
 depends="python-pycparser"
-checkdepends="python3-pytest python3-pycparser"
 short_desc="C foreign function interface for Python2"
 maintainer="Andrew J. Hesford <ajh@sideband.org>"
 license="MIT"
@@ -15,28 +14,6 @@ changelog="https://cffi.readthedocs.io/en/latest/whatsnew.html"
 distfiles="${PYPI_SITE}/c/cffi/cffi-${version}.tar.gz"
 checksum=d400bfb9a37b1351253cb402671cea7e89bdecc294e8016a707f6d1d8ac934f9
 
-do_check() {
-	# glibc libm.so is a GNU ld script that isn't properly handled
-	# on dlopen; libm symbols required in these tests are missing
-	local excludes='not sin'
-	excludes+=' and not test_dlopen'
-	excludes+=' and not test_function_typedef'
-	excludes+=' and not test_wraps_from_stdlib'
-	excludes+=' and not test_stdcall_only_on_windows'
-
-	PYTHONPATH="$(cd build-${py3_ver}/lib* && pwd)" \
-		python3 -m pytest c/ testing/ -x -k "$excludes"
-}
-
 post_install() {
 	vlicense LICENSE
 }
-
-python3-cffi_package() {
-	depends="python3-pycparser"
-	short_desc="${short_desc/Python2/Python3}"
-	pkg_install() {
-		vmove usr/lib/python3*
-		vlicense LICENSE
-	}
-}
diff --git a/srcpkgs/python3-cffi b/srcpkgs/python3-cffi
deleted file mode 120000
index 8d25d17c4b33..000000000000
--- a/srcpkgs/python3-cffi
+++ /dev/null
@@ -1 +0,0 @@
-python-cffi
\ No newline at end of file
diff --git a/srcpkgs/python3-cffi/template b/srcpkgs/python3-cffi/template
new file mode 100644
index 000000000000..9be8775ca6d5
--- /dev/null
+++ b/srcpkgs/python3-cffi/template
@@ -0,0 +1,33 @@
+# Template file for 'python3-cffi'
+pkgname=python3-cffi
+version=1.15.1
+revision=2
+build_style=python3-module
+hostmakedepends="python3-setuptools libffi-devel"
+makedepends="python3-devel libffi-devel"
+depends="python3-pycparser"
+checkdepends="python3-pytest ${depends}"
+short_desc="C foreign function interface for Python3"
+maintainer="Andrew J. Hesford <ajh@sideband.org>"
+license="MIT"
+homepage="https://cffi.readthedocs.io/"
+changelog="https://cffi.readthedocs.io/en/latest/whatsnew.html"
+distfiles="${PYPI_SITE}/c/cffi/cffi-${version}.tar.gz"
+checksum=d400bfb9a37b1351253cb402671cea7e89bdecc294e8016a707f6d1d8ac934f9
+
+do_check() {
+	# glibc libm.so is a GNU ld script that isn't properly handled
+	# on dlopen; libm symbols required in these tests are missing
+	local excludes='not sin'
+	excludes+=' and not test_dlopen'
+	excludes+=' and not test_function_typedef'
+	excludes+=' and not test_wraps_from_stdlib'
+	excludes+=' and not test_stdcall_only_on_windows'
+
+	PYTHONPATH="$(cd build/lib* && pwd)" \
+		python3 -m pytest c/ testing/ -x -k "$excludes"
+}
+
+post_install() {
+	vlicense LICENSE
+}

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

* Re: [PR PATCH] [Merged]: python-cffi: split python3-cffi to a new package
  2023-06-16 21:34 [PR PATCH] python-cffi: split python3-cffi to a new package paper42
@ 2023-06-17 17:04 ` paper42
  0 siblings, 0 replies; 2+ messages in thread
From: paper42 @ 2023-06-17 17:04 UTC (permalink / raw)
  To: ml

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

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

python-cffi: split python3-cffi to a new package
https://github.com/void-linux/void-packages/pull/44483

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

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

@ahesford 
<!--
#### 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] 2+ messages in thread

end of thread, other threads:[~2023-06-17 17:04 UTC | newest]

Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-06-16 21:34 [PR PATCH] python-cffi: split python3-cffi to a new package paper42
2023-06-17 17:04 ` [PR PATCH] [Merged]: " paper42

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