Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] python3: force sem_gevalue for all cross arches
@ 2023-05-16 14:37 dkwo
  2023-05-18 16:04 ` [PR PATCH] [Merged]: " ahesford
  2023-05-18 16:04 ` ahesford
  0 siblings, 2 replies; 3+ messages in thread
From: dkwo @ 2023-05-16 14:37 UTC (permalink / raw)
  To: ml

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

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

https://github.com/dkwo/void-packages py3
https://github.com/void-linux/void-packages/pull/43902

python3: force sem_gevalue for all cross arches
- I tested the changes in this PR: yes
- I built this PR locally for my native architecture, (aarch64)

As discussed in https://github.com/void-linux/void-packages/pull/43659 this is checked by compiling and running a program that won't work in cross, but we know `sem_gevalue` is ok at least on `aarch64` (to be more conservative, I could also only enable it there, rather than all arches).

cc @ahesford 

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

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

From 4f81c2fe59c2f96c568d99454e85f9f2d316a0c1 Mon Sep 17 00:00:00 2001
From: dkwo <nicolopiazzalunga@gmail.com>
Date: Tue, 16 May 2023 10:32:42 -0400
Subject: [PATCH] python3: force sem_gevalue for all cross arch

---
 srcpkgs/python3/template | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/srcpkgs/python3/template b/srcpkgs/python3/template
index aebf41852764..04ef3591be55 100644
--- a/srcpkgs/python3/template
+++ b/srcpkgs/python3/template
@@ -4,7 +4,7 @@
 #
 pkgname=python3
 version=3.11.3
-revision=1
+revision=2
 build_style="gnu-configure"
 configure_args="--enable-shared --enable-ipv6
  --enable-loadable-sqlite-extensions --with-computed-gotos
@@ -39,6 +39,7 @@ alternatives="
 if [ "$CROSS_BUILD" ]; then
 	hostmakedepends+=" python3"
 	configure_args+=" --with-build-python=python3.11"
+	configure_args+=" ac_cv_broken_sem_getvalue=no"
 fi
 
 post_extract() {

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

* Re: [PR PATCH] [Merged]: python3: force sem_gevalue for all cross arches
  2023-05-16 14:37 [PR PATCH] python3: force sem_gevalue for all cross arches dkwo
@ 2023-05-18 16:04 ` ahesford
  2023-05-18 16:04 ` ahesford
  1 sibling, 0 replies; 3+ messages in thread
From: ahesford @ 2023-05-18 16:04 UTC (permalink / raw)
  To: ml

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

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

python3: force sem_gevalue for all cross arches
https://github.com/void-linux/void-packages/pull/43902

Description:
- I tested the changes in this PR: yes
- I built this PR locally for my native architecture, (aarch64)

As discussed in https://github.com/void-linux/void-packages/pull/43659 this is checked by compiling and running a program that won't work in cross, but we know `sem_gevalue` is ok at least on `aarch64` (to be more conservative, I could also only enable it there, rather than all arches).

cc @ahesford 

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

* Re: python3: force sem_gevalue for all cross arches
  2023-05-16 14:37 [PR PATCH] python3: force sem_gevalue for all cross arches dkwo
  2023-05-18 16:04 ` [PR PATCH] [Merged]: " ahesford
@ 2023-05-18 16:04 ` ahesford
  1 sibling, 0 replies; 3+ messages in thread
From: ahesford @ 2023-05-18 16:04 UTC (permalink / raw)
  To: ml

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

New comment by ahesford on void-packages repository

https://github.com/void-linux/void-packages/pull/43902#issuecomment-1553279852

Comment:
Let's assume it works everywhere and disable it if somebody on `arm*` notices otherwise.

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

end of thread, other threads:[~2023-05-18 16:04 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-05-16 14:37 [PR PATCH] python3: force sem_gevalue for all cross arches dkwo
2023-05-18 16:04 ` [PR PATCH] [Merged]: " ahesford
2023-05-18 16:04 ` ahesford

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