From: fosslinux <fosslinux@users.noreply.github.com>
To: ml@inbox.vuxu.org
Subject: [PR PATCH] gnubg: fix arm, cleanup template.
Date: Sat, 10 Oct 2020 00:42:16 +0200 [thread overview]
Message-ID: <gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-25483@inbox.vuxu.org> (raw)
[-- 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
}
-
next reply other threads:[~2020-10-09 22:42 UTC|newest]
Thread overview: 3+ messages / expand[flat|nested] mbox.gz Atom feed top
2020-10-09 22:42 fosslinux [this message]
2020-10-09 23:37 ` sgn
2020-10-09 23:37 ` [PR PATCH] [Closed]: " sgn
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=gh-mailinglist-notifications-41a7ca26-5023-4802-975b-f1789d68868e-void-packages-25483@inbox.vuxu.org \
--to=fosslinux@users.noreply.github.com \
--cc=ml@inbox.vuxu.org \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).