Github messages for voidlinux
 help / color / mirror / Atom feed
* [PR PATCH] gnubg: fix arm, cleanup template.
@ 2020-10-09 22:42 fosslinux
  2020-10-09 23:37 ` sgn
  2020-10-09 23:37 ` [PR PATCH] [Closed]: " sgn
  0 siblings, 2 replies; 3+ messages in thread
From: fosslinux @ 2020-10-09 22:42 UTC (permalink / raw)
  To: ml

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

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

https://github.com/fosslinux/void-packages gnubg-cleanup
https://github.com/void-linux/void-packages/pull/25483

gnubg: fix arm, cleanup template.
ARM was failing because it looks for python >=2.1.0. However, it
dosen't know that py3 exists and is very different. Hence, it tries
to use python 3.8 - which dosen't work. Setting CFLAGS and LDFLAGS
so that it uses python2.7 makes the build work.

Furthermore, python-devel in hostmakedepends confused it somewhat,
hence removed (it is not required), and python is required at runtime
(shlibs should pick that up, but it /is/ an explicit dependency
for scripting, and if the python package structure changes then
this is required).

Template cleanups:

- fix xlint warnings.
- move if statements below main variables.
- move the homepage to the new GNU location, gnubg.org is dead.
- move distfiles to working location.

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

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

From 2649cd9d7102c227bbbc3545313459bae4728d5e Mon Sep 17 00:00:00 2001
From: fosslinux <fosslinux@aussies.space>
Date: Fri, 9 Oct 2020 20:22:08 +1100
Subject: [PATCH] gnubg: fix arm, cleanup template.

ARM was failing because it looks for python >=2.1.0. However, it
dosen't know that py3 exists and is very different. Hence, it tries
to use python 3.8 - which dosen't work. Setting CFLAGS and LDFLAGS
so that it uses python2.7 makes the build work.

Furthermore, python-devel in hostmakedepends confused it somewhat,
hence removed (it is not required), and python is required at runtime
(shlibs should pick that up, but it /is/ an explicit dependency
for scripting, and if the python package structure changes then
this is required).

Template cleanups:

- fix xlint warnings.
- move if statements below main variables.
- move the homepage to the new GNU location, gnubg.org is dead.
- move distfiles to working location.
---
 srcpkgs/gnubg/template | 29 ++++++++++++++---------------
 1 file changed, 14 insertions(+), 15 deletions(-)

diff --git a/srcpkgs/gnubg/template b/srcpkgs/gnubg/template
index 4bbe300c6c6..76f01d08d4a 100644
--- a/srcpkgs/gnubg/template
+++ b/srcpkgs/gnubg/template
@@ -1,29 +1,29 @@
 # Template file for 'gnubg'
 pkgname=gnubg
 version=1.06.002
-revision=2
+revision=3
 build_style=gnu-configure
 configure_args="--disable-cputest ax_cv_gcc_x86_cpuid_0x00000001=6000000"
-if [ "$XBPS_TARGET_MACHINE" = x86_64 ]; then
-	configure_args+=" --enable-simd=sse2"
-fi
-hostmakedepends="pkg-config python-devel gdk-pixbuf-devel"
-if [ "$CROSS_BUILD" ]; then
-	hostmakedepends+=" ${pkgname}"
-fi
+hostmakedepends="pkg-config gdk-pixbuf-devel"
 makedepends="libcurl-devel libglib-devel gtk+-devel gtkglext-devel python-devel
  libcanberra-devel readline-devel readline sqlite-devel"
+depends="python"
 short_desc="GNU Backgammon game"
 maintainer="Leah Neukirchen <leah@vuxu.org>"
 license="GPL-3.0-or-later"
-homepage="http://www.gnubg.org"
-distfiles="http://www.gnubg.org/media/sources/$pkgname-release-$version-sources.tar.gz"
+homepage="https://www.gnu.org/software/gnubg"
+distfiles="${GNU_SITE}/${pkgname}/${pkgname}-release-${version}-sources.tar.gz"
 checksum=ce1b0b0c1900717cc598032a14cf8c0ee60faf24d84368b39922c0102983bc87
-LDFLAGS="-Wl,--no-as-needed -lm -lreadline"
+LDFLAGS+=" -Wl,--no-as-needed -lm -lreadline -lpython2.7 -I${XBPS_CROSS_BASE}/usr/include/python2.7"
+CFLAGS+=" -I${XBPS_CROSS_BASE}/usr/include/python2.7"
+
+if [ "$XBPS_TARGET_MACHINE" = x86_64 ]; then
+	configure_args+=" --enable-simd=sse2"
+fi
 
-case "$XBPS_TARGET_MACHINE" in
-	arm*) broken="https://build.voidlinux.org/builders/armv7l_builder/builds/6536/steps/shell_3/logs/stdio" ;;
-esac
+if [ "$CROSS_BUILD" ]; then
+	hostmakedepends+=" ${pkgname}"
+fi
 
 pre_configure() {
 	if [ "$CROSS_BUILD" ]; then
@@ -32,4 +32,3 @@ pre_configure() {
 		makebearoff -t 6x6 -f gnubg_ts0.bd
 	fi
 }
-

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

* Re: gnubg: fix arm, cleanup template.
  2020-10-09 22:42 [PR PATCH] gnubg: fix arm, cleanup template fosslinux
@ 2020-10-09 23:37 ` sgn
  2020-10-09 23:37 ` [PR PATCH] [Closed]: " sgn
  1 sibling, 0 replies; 3+ messages in thread
From: sgn @ 2020-10-09 23:37 UTC (permalink / raw)
  To: ml

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

New comment by sgn on void-packages repository

https://github.com/void-linux/void-packages/pull/25483#issuecomment-706444390

Comment:
I'll meld this into https://github.com/void-linux/void-packages/pull/25455

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

* Re: [PR PATCH] [Closed]: gnubg: fix arm, cleanup template.
  2020-10-09 22:42 [PR PATCH] gnubg: fix arm, cleanup template fosslinux
  2020-10-09 23:37 ` sgn
@ 2020-10-09 23:37 ` sgn
  1 sibling, 0 replies; 3+ messages in thread
From: sgn @ 2020-10-09 23:37 UTC (permalink / raw)
  To: ml

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

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

gnubg: fix arm, cleanup template.
https://github.com/void-linux/void-packages/pull/25483

Description:
ARM was failing because it looks for python >=2.1.0. However, it
dosen't know that py3 exists and is very different. Hence, it tries
to use python 3.8 - which dosen't work. Setting CFLAGS and LDFLAGS
so that it uses python2.7 makes the build work.

Furthermore, python-devel in hostmakedepends confused it somewhat,
hence removed (it is not required), and python is required at runtime
(shlibs should pick that up, but it /is/ an explicit dependency
for scripting, and if the python package structure changes then
this is required).

Template cleanups:

- fix xlint warnings.
- move if statements below main variables.
- move the homepage to the new GNU location, gnubg.org is dead.
- move distfiles to working location.

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

end of thread, other threads:[~2020-10-09 23:37 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-10-09 22:42 [PR PATCH] gnubg: fix arm, cleanup template fosslinux
2020-10-09 23:37 ` sgn
2020-10-09 23:37 ` [PR PATCH] [Closed]: " sgn

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