sci-mathematics/ggnfs: Respect CC and CFLAGS and link properly respecting LDFLAGS, #451046

Package-Manager: portage-2.2.0_alpha166/cvs/Linux x86_64
Manifest-Sign-Key: 0x70EB7916

5 files changed, 209 insertions, 20 deletions

diff --git a/sci-mathematics/ggnfs/ChangeLog b/sci-mathematics/ggnfs/ChangeLog
index 25af661f0cc7..8504913c62d2 100644
--- a/sci-mathematics/ggnfs/ChangeLog
+++ b/sci-mathematics/ggnfs/ChangeLog
@@ -1,6 +1,12 @@
 # ChangeLog for sci-mathematics/ggnfs
-# Copyright 1999-2012 Gentoo Foundation; Distributed under the GPL v2
-# $Header: /var/cvsroot/gentoo-x86/sci-mathematics/ggnfs/ChangeLog,v 1.5 2012/11/30 08:00:06 patrick Exp $
+# Copyright 1999-2013 Gentoo Foundation; Distributed under the GPL v2
+# $Header: /var/cvsroot/gentoo-x86/sci-mathematics/ggnfs/ChangeLog,v 1.6 2013/03/07 08:34:59 jlec Exp $
+
+*ggnfs-0.77.1-r2 (07 Mar 2013)
+
+  07 Mar 2013; Justin Lecher <jlec@gentoo.org> -ggnfs-0.77.1-r1.ebuild,
+  +ggnfs-0.77.1-r2.ebuild, +files/ggnfs-0.77.1-gentoo.patch, metadata.xml:
+  Respect CC and CFLAGS and link properly respecting LDFLAGS, #451046
 
   30 Nov 2012; Patrick Lauer <patrick@gentoo.org> ggnfs-0.77.1-r1.ebuild:
   Correctify displayed version number
diff --git a/sci-mathematics/ggnfs/Manifest b/sci-mathematics/ggnfs/Manifest
index f2ea5cbbb999..2f55035ff15f 100644
--- a/sci-mathematics/ggnfs/Manifest
+++ b/sci-mathematics/ggnfs/Manifest
@@ -1,6 +1,17 @@
+-----BEGIN PGP SIGNED MESSAGE-----
+Hash: SHA256
+
+AUX ggnfs-0.77.1-gentoo.patch 5074 SHA256 7e303c43095abb0b4ace6fb1f9f6c790cf57f93afb568a22f171133daf52d339 SHA512 bc2dc58784aa5c78b79f5021f57f7974dbb475e2489d8864549134dbf002aefd7546d2eadca03d04daa89638de02494052583e91b4b79628b500a1c584dd3f0a WHIRLPOOL 2253fec0d4eaa8b80d8a42cb1302d372e1d95d8d24fe293422ff5b2d0ba83ed7dc2f14d0e419b2b4ab1ff4c13074f0b1839d586a9dd22198b63329be1870ead1
 DIST def-nm-params.txt 3495 SHA256 5febf991cdd4112265bd58379c6d0819e6bb01b6e656902256fe13e4b63ba97c SHA512 b924292bc5354f87a3f88f2c35b767b9183704ced61c0e709cdd0974a2e1f34a4c170c6d12524b10162f63306f74001ad30e82ac7997d98db823006086772460 WHIRLPOOL 9ecabb6bee118269bd4d345582d36f9f82d54585ce7092adb3b6e1d27be4d0297201a408c76bb692047192a996982113d8387aad3c4764062f6d23f2f9febc82
 DIST def-par.txt 4957 SHA256 40d0b9be2c52282f29ff8cbc50799b625fc05b1da6eb7f2070fb3eb2ac81e7fa SHA512 fc0851513b1170e0a43e7458e24da4274146272018b7fefbe35280005ebeba20458756c4d4715b2a98780354dd1c5bcf31d98dd1388f1d352fb718062141c434 WHIRLPOOL 81151ab8235b2b55a7e8ceca4453b49bab3d3109eaf1352808b7e42c3612977402856c518ba8d8a1e5b6b80e33bbc3f33bdc565da5d3ee9e4251a1d253ace496
 DIST ggnfs-0.77.1.zip 2279013 SHA256 f6509661f22c562b3616e99989d39c160e8f49664e1fe3215d2f0d1441003a7a SHA512 77f5050d1310df5de45e80a25c8421cc9240ce11776fc4af596ce19d8b064b926dcc7e6dd7b2ef12d86056f7af0b67db2f746fc261cba24bbcf1efef06287513 WHIRLPOOL 5cb0e67aebd7ed9001ea2347b2211f9ca84ec50d18982436b111c0abb1c3223c66fbb5d029fa38d43e1137e7e7c7087daca85648fba7da92bbfcfe3760ccf87e
-EBUILD ggnfs-0.77.1-r1.ebuild 2362 SHA256 7ee663a6458fe911b1f4be837e8d8a4ac95571a52d341204acefbd5f38b17192 SHA512 74f2edc4a9c7bb2953c96f914e5d8b6fa2b1c614b6896814345f3c137acbe7884411a37a8d59f731ae3ccdfa6488e80bf7558a88bf650439f7a152dce1957e5d WHIRLPOOL 9df8320e10f451287ca911ff4032f4a8545f478efceaf82433535e1f40bf5635216557277e2be750710edf4b2f4e22c5737f4e0226f93ad4acd2038a6d938fb5
-MISC ChangeLog 869 SHA256 0234c93e1fd5701df7a12c95aac5f4fa0c4daf7f90ed8d0c9691217ecbcef482 SHA512 9f7901c1491c436c52c1a220eb13209d973fdb6b24e10bd5a88ffb294bd2c17fa4fb32dbd70f586ac24f6f83c5160cc7f39613c1447ea78bbc1ca0069da097d5 WHIRLPOOL 0a6d16a578129d212bbd12c11544d1562602b55a02545f2acee39e2e52c1e6caad0be395fd47e3f035ac5bdf07bb4e77f5236f8bb2afbcfe151d07327000d607
-MISC metadata.xml 233 SHA256 65209e280ca6de32a92e1a57301ff69fb3472e55ba07848d6f80405d35e92d47 SHA512 0d30bfb39315a04767e07896409f5d3954f8e954e20fcbac7b92bb9a636c9f61c50b1b2cf55e6c1db9a199a0edc159b5c8cc8f541efb9c4adafa28e3652cff7b WHIRLPOOL 027c5101e1cbc665c9d2ef240f73a874c9f49a9b70175c5947e012df61de484585088a46b6ce9078816e548e6bcc326e7223539cc999159eee2757d0a2eb1480
+EBUILD ggnfs-0.77.1-r2.ebuild 2656 SHA256 a550be88ad325fe958be19b5b57ab2e456bd2ee81ad8340f0de7deb862d10a29 SHA512 c377a72a509fe3dc1ac0af9169ae54a74a0dda130e23ad7fc4d143243a2d2dc18dfc5e0c174a17e40fc0d139dbd94a1402fe8534ce5e84b0360bb9d2526c8c74 WHIRLPOOL 472f062b67f27f0b5c7183ff80060774252e334c0387d350f9765d2901aea83233451a04c0b82402f6c3869afdb18ec0ae020c25259858795dd5a0655508f2a5
+MISC ChangeLog 1116 SHA256 92c105fe19f358a07dccde7e07cf141ccad1b302157ba4c693cd68368e84d71f SHA512 7dd2e5e43068fe88c73184f20ae4ef6537de5978f68b10ca7df6fb51f9ccbd745fac7bb093afa0124c43460148e1b4cdaed635f81bd5d8299f430dede83f69ca WHIRLPOOL e1b00a15a1ca55c2dd84bcce89139e3cfd36eef63246be648ee5a98bef8f4e593ba8cfeb6698f1321982753ce472b309f5be09ffb189781c57cc79b80b711df9
+MISC metadata.xml 239 SHA256 57c79f6c1b1b06b8d90265f1cc0affe89b5548b63e811f0313bfb2a0a0aa201a SHA512 1898d4e8bb4f02ffdad555a095ee76f934bd6ee1f914564eac1932f0dd96f26a52f47047324ed0a81009516a84bf72e1b10922fa751f322abf17517f84f62156 WHIRLPOOL 7375ace29b0d9c7e945e141c09b70c79599780bc9d3478e59fbc85dc4330aeda6bf2bb54fde24d25feb181f88618e11e022a90674d1f26ff9ef692bd698ff754
+-----BEGIN PGP SIGNATURE-----
+Version: GnuPG v2.0.19 (GNU/Linux)
+
+iEYEAREIAAYFAlE4UTQACgkQgAnW8HDreRaaoQCdH+xWb5M/1CJj5vd04SMDZwU7
+wAEAnjSEx8ewThDmfKiHN570V/Y023mD
+=gRTV
+-----END PGP SIGNATURE-----
diff --git a/sci-mathematics/ggnfs/files/ggnfs-0.77.1-gentoo.patch b/sci-mathematics/ggnfs/files/ggnfs-0.77.1-gentoo.patch
new file mode 100644
index 000000000000..ac7516034147
--- /dev/null
+++ b/sci-mathematics/ggnfs/files/ggnfs-0.77.1-gentoo.patch
@@ -0,0 +1,158 @@
+ src/Makefile          | 34 +++++++++++++++++-----------------
+ src/lasieve4/Makefile |  6 +++---
+ src/pol5/Makefile     |  8 ++++----
+ 3 files changed, 24 insertions(+), 24 deletions(-)
+
+diff --git a/src/Makefile b/src/Makefile
+index 01f66be..59b433f 100644
+--- a/src/Makefile
++++ b/src/Makefile
+@@ -1,15 +1,15 @@
+ # Workaround for PPC
+ # Due to unknown reason -march isn't supported there
+ ifeq ($(ARCH),970)
+-    ALLOPT=-mcpu=$(ARCH) -m64 -pipe
++    ALLOPT=
+ else
+-    ALLOPT=-march=$(ARCH) -pipe
++    ALLOPT=
+ endif
+ 
+ INC=-I. -I.. -I../include $(LOCALINC)
+ LIBS=-lgmp -lm
+ BINDIR=../bin
+-LIBFLAGS=$(LOCALLIB)
++LDFLAGS+=$(LOCALLIB)
+ 
+ MATBUILD_TPIE=0
+ TPIE_DIR=../../tpie
+@@ -17,7 +17,7 @@ TPIE_DIR=../../tpie
+ # Turn on\off workaround for GMP 4.2 bug
+ # Consult http://swox.com/list-archives/gmp-bugs/2006-May/000475.html
+ # for more info
+-GMP_BUG=1
++GMP_BUG=0
+ 
+ ifeq ($(HOST),generic)
+     ALLOPT+= -DGGNFS_HOST_GENERIC
+@@ -32,10 +32,10 @@ ifeq ($(ARCH),prescott)
+     ALLOPT+= -DL2_CACHE_SIZE=1024
+ endif
+ 
+-DEBUGOPT=-W -Wall -g -pg
++DEBUGOPT=-W -Wall
+ 
+-CFLAGS=-DNDEBUG -O3 -funroll-loops -finline-functions \
+-       -ftracer -fomit-frame-pointer -W -Wall $(ALLOPT)
++CFLAGS+=-DNDEBUG \
++       -W -Wall $(ALLOPT)
+ # CFLAGS=$(DEBUGOPT) $(ALLOPT)
+ # -ffast-math removed since -funsafe-math-optimizations seems to cause occasional problems, especially in sqrt
+ 
+@@ -71,7 +71,7 @@ endif
+ ifeq ($(MATBUILD_TPIE),1)
+   BINS+= $(BINDIR)/matbuild-tpie
+   INC+= -I$(TPIE_DIR)/include
+-  LIBFLAGS+= -L$(TPIE_DIR)/lib
++  LDFLAGS+= -L$(TPIE_DIR)/lib
+   LIBS+=-ltpie
+ endif
+ 
+@@ -97,35 +97,35 @@ bins : $(BINS)
+ 	$(CC) $(CFLAGS) -o $@ -c $*.S
+ 
+ $(BINDIR)/sieve : sieve.c clsieve.c $(OBJS) makefb.o
+-	$(CC) $(INC) $(CFLAGS) $(LIBFLAGS) -o $@ $^ $(LIBS)
++	$(CC) $(INC) $(CFLAGS) $(LDFLAGS) -o $@ $^ $(LIBS)
+ 
+ $(BINDIR)/makefb : makefb.c $(OBJS)
+-	$(CC) $(INC) $(CFLAGS) $(LIBFLAGS) -D_MAKEFB_STANDALONE -o $@ \
++	$(CC) $(INC) $(CFLAGS) $(LDFLAGS) -D_MAKEFB_STANDALONE -o $@ \
+ 	makefb.c $(OBJS) $(LIBS)
+ 
+ $(BINDIR)/procrels : procrels.c $(OBJS)
+-	$(CC) $(INC) $(CFLAGS) $(LIBFLAGS) -o $@ procrels.c $(OBJS) $(LIBS)
++	$(CC) $(INC) $(CFLAGS) $(LDFLAGS) -o $@ procrels.c $(OBJS) $(LIBS)
+ 
+ $(BINDIR)/matbuild : matbuild.c $(OBJS)
+-	$(CC) $(INC) $(CFLAGS) $(LIBFLAGS) -o $@ matbuild.c combparts.c \
++	$(CC) $(INC) $(CFLAGS) $(LDFLAGS) -o $@ matbuild.c combparts.c \
+ 	  $(OBJS) $(LIBS)
+ 
+ $(BINDIR)/matbuild-tpie : matbuild.c $(OBJS)
+-	$(CPP) $(INC) $(CFLAGS) $(LIBFLAGS) -DGGNFS_TPIE -o $@ matbuild.c \
++	$(CPP) $(INC) $(CFLAGS) $(LDFLAGS) -DGGNFS_TPIE -o $@ matbuild.c \
+ 	  experimental/combparts_tpie.cpp experimental/llist_tpie.cpp \
+ 	  $(OBJS) $(LIBS)
+ 
+ $(BINDIR)/matprune : matprune.c $(OBJS)
+-	$(CC) $(INC) $(CFLAGS) $(LIBFLAGS) -o $@ matprune.c $(OBJS) $(LIBS)
++	$(CC) $(INC) $(CFLAGS) $(LDFLAGS) -o $@ matprune.c $(OBJS) $(LIBS)
+ 
+ $(BINDIR)/matsolve : matsolve.c $(OBJS)
+-	$(CC) $(INC) $(CFLAGS) $(LIBFLAGS) -o $@ matsolve.c $(OBJS) $(LIBS)
++	$(CC) $(INC) $(CFLAGS) $(LDFLAGS) -o $@ matsolve.c $(OBJS) $(LIBS)
+ 
+ $(BINDIR)/sqrt : sqrt.c $(OBJS)
+-	$(CC) $(INC) $(CFLAGS) $(LIBFLAGS) -o $@ sqrt.c $(OBJS) $(LIBS)
++	$(CC) $(INC) $(CFLAGS) $(LDFLAGS) -o $@ sqrt.c $(OBJS) $(LIBS)
+ 
+ $(BINDIR)/polyselect : polyselect.c $(OBJS)
+-	$(CC) $(INC) $(CFLAGS) $(LIBFLAGS) -o $@ polyselect.c $(OBJS) $(LIBS)
++	$(CC) $(INC) $(CFLAGS) $(LDFLAGS) -o $@ polyselect.c $(OBJS) $(LIBS)
+ 
+ latsiever :
+ 	$(MAKE) -C lasieve4
+diff --git a/src/lasieve4/Makefile b/src/lasieve4/Makefile
+index 31e05a9..6886b16 100644
+--- a/src/lasieve4/Makefile
++++ b/src/lasieve4/Makefile
+@@ -10,8 +10,8 @@
+ # 6/13/04: Hacked up for use in GGNFS by Chris Monico.
+ 
+ INC=-I.. -I../../include $(LOCALINC)
+-LIBFLAGS=-L. $(LOCALLIB)
+-CFLAGS+=-Os
++LDFLAGS+=-L. $(LOCALLIB)
++CFLAGS+=
+ 
+ BINDIR=../../bin
+ FACT=prho.o
+@@ -58,7 +58,7 @@ gnfs-lasieve4eI%.o: gnfs-lasieve4e.c lasieve.h asm/siever-config.h
+ 
+ $(BINDIR)/gnfs-lasieve4I%e: gnfs-lasieve4eI%.o $(OBJS) libgmp-aux.a \
+                              asm/liblasieve.a asm/liblasieveI%.a $(FACT)
+-	$(CC) $(CFLAGS) $(INC) $(LIBFLAGS) -o $@ $^ $(LIBS)
++	$(CC) $(CFLAGS) $(INC) $(LDFLAGS) -o $@ $^ $(LIBS)
+ 
+ asm/lib%.a:
+ 	$(MAKE) -C asm
+diff --git a/src/pol5/Makefile b/src/pol5/Makefile
+index 44b08be..e474ffb 100644
+--- a/src/pol5/Makefile
++++ b/src/pol5/Makefile
+@@ -1,6 +1,6 @@
+ BINDIR=../../bin
+ INC=-I.. -I../../include $(LOCALINC)
+-LIBFLAGS=$(LOCALLIB)
++LDFLAGS+=$(LOCALLIB)
+ LIBS=-lm -lgmp
+ CFLAGS2=
+ 
+@@ -46,13 +46,13 @@ all: $(BINS)
+ 	$(CC) $(INC) -c $(CFLAGS) $(CFLAGS2) $^
+ 
+ $(BINDIR)/pol51m0b: pol51m0b.o $(OBJS) $(OBJS2)
+-	$(CC) $(INC) -o $@ $(CFLAGS) $(CFLAGS2) $^ $(LIBFLAGS) $(LIBS)
++	$(CC) $(INC) $(LDFLAGS) -o $@ $(CFLAGS) $(CFLAGS2) $^ $(LIBS)
+ 
+ $(BINDIR)/pol51m0n: pol51m0n.o $(OBJS) $(OBJS3)
+-	$(CC) $(INC) $(CFLAGS) $(CFLAGS2) -o $@ $^ $(LIBFLAGS) $(LIBS)
++	$(CC) $(INC) $(CFLAGS) $(CFLAGS2) $(LDFLAGS) -o $@ $^ $(LIBS)
+ 
+ $(BINDIR)/pol51opt: pol51opt.o $(OBJS) $(OBJS4)
+-	$(CC) $(INC) $(CFLAGS) $(CFLAGS2) -o $@ $^ $(LIBFLAGS) $(LIBS)
++	$(CC) $(INC) $(CFLAGS) $(CFLAGS2) $(LDFLAGS) -o $@ $^ $(LIBS)
+ 
+ clean : ;
+ 	rm -f $(BINS) *.o core
diff --git a/sci-mathematics/ggnfs/ggnfs-0.77.1-r1.ebuild b/sci-mathematics/ggnfs/ggnfs-0.77.1-r2.ebuild
index 6de63de483b1..0019df5d801f 100644
--- a/sci-mathematics/ggnfs/ggnfs-0.77.1-r1.ebuild
+++ b/sci-mathematics/ggnfs/ggnfs-0.77.1-r2.ebuild
@@ -1,32 +1,45 @@
-# Copyright 1999-2012 Gentoo Foundation
+# Copyright 1999-2013 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/sci-mathematics/ggnfs/ggnfs-0.77.1-r1.ebuild,v 1.3 2012/11/30 08:00:06 patrick Exp $
+# $Header: /var/cvsroot/gentoo-x86/sci-mathematics/ggnfs/ggnfs-0.77.1-r2.ebuild,v 1.1 2013/03/07 08:34:59 jlec Exp $
+
+EAPI=5
+
+inherit eutils
 
-EAPI=4
 DESCRIPTION="A suite of algorithms to help factoring large integers"
 # inactive old homepage exists, this is a fork
 HOMEPAGE="https://github.com/radii/ggnfs"
 # snapshot because github makes people stupid
-SRC_URI="http://dev.gentooexperimental.org/~dreeevil/${P}.zip
+SRC_URI="
+	http://dev.gentooexperimental.org/~dreeevil/${P}.zip
 	http://stuff.mit.edu/afs/sipb/project/pari-gp/ggnfs/Linux/src/def-par.txt
 	http://stuff.mit.edu/afs/sipb/project/pari-gp/ggnfs/Linux/src/def-nm-params.txt"
 
-inherit eutils
-
-LICENSE="GPL-2"
 SLOT="0"
-# Need to test if it actually compiles on x86
+LICENSE="GPL-2"
 KEYWORDS="~amd64 ~x86"
 IUSE=""
 
-DEPEND=""
+DEPEND=">=dev-libs/gmp-4.3:0"
 RDEPEND="${DEPEND}
 	!sci-mathematics/cado-nfs" # file collisions, fixable
 
 S=${WORKDIR}/${PN}-master
 
+pkg_setup() {
+	einfo "There are several internal tuning options"
+	einfo "Please export ARCH being on of the following"
+	einfo "	prescott"
+	einfo "	pentium2"
+	einfo "	pentium4"
+	einfo "if your cpu is of that type"
+
+	[[ -z ${ARCH} ]] && export ARCH=generic
+}
+
 src_prepare() {
-	echo "#define GGNFS_VERSION \"0.77.1-$ARCH\"" > include/version.h
+	epatch "${FILESDIR}"/${P}-gentoo.patch
+	echo "#define GGNFS_VERSION \"0.77.1-$ARCH\"" > include/version.h || die
 	# fix directory symlink, add missing targets, rewrite variable used by portage internally
 	cd src/lasieve4 && rm -f -r asm && ln -s ppc32 asm || die
 	sed -i -e 's/all: liblasieve.a/all: liblasieve.a liblasieveI11.a liblasieveI15.a liblasieveI16.a/' asm/Makefile || die
@@ -34,6 +47,7 @@ src_prepare() {
 	sed -i -e 's/ARCH/MARCH/g' Makefile src/Makefile || die
 	sed -i -e 's/$(LSBINS) strip/$(LSBINS)/' src/Makefile || die #No stripping!
 	sed -i -e 's/SVN \$Revision\$/0.77.1 snapshot/' src/experimental/lasieve4_64/gnfs-lasieve4e.c src/lasieve4/gnfs-lasieve4e.c || die
+	tc-export CC
 }
 
 src_configure() { :; }
@@ -42,7 +56,7 @@ src_compile() {
 	# setting MARCH like this is fugly, but it uses -march=$ARCH - better fix welcome
 	# it also assumes a recent-ish compiler
 	cd src
-	HOST="generic" MARCH="native" emake -j1
+	HOST="generic" MARCH="${ARCH}" emake -j1
 }
 
 src_install() {
diff --git a/sci-mathematics/ggnfs/metadata.xml b/sci-mathematics/ggnfs/metadata.xml
index 24cbb90cf0c4..6719fcd2bb35 100644
--- a/sci-mathematics/ggnfs/metadata.xml
+++ b/sci-mathematics/ggnfs/metadata.xml
@@ -1,8 +1,8 @@
 <?xml version="1.0" encoding="UTF-8"?>
 <!DOCTYPE pkgmetadata SYSTEM "http://www.gentoo.org/dtd/metadata.dtd">
 <pkgmetadata>
-	<maintainer>
-		<email>patrick@gentoo.org</email>
-		<name>Patrick Lauer</name>
-	</maintainer>
+  <maintainer>
+    <email>patrick@gentoo.org</email>
+    <name>Patrick Lauer</name>
+  </maintainer>
 </pkgmetadata>