Use CC and LDFLAGS and fix building with -Wl,--as-needed wrt #277031

Package-Manager: portage-2.2_rc33/cvs/Linux x86_64

4 files changed, 124 insertions, 31 deletions

diff --git a/app-misc/mplinuxman/ChangeLog b/app-misc/mplinuxman/ChangeLog
index cd9b99eb8a91..e323617b3509 100644
--- a/app-misc/mplinuxman/ChangeLog
+++ b/app-misc/mplinuxman/ChangeLog
@@ -1,6 +1,10 @@
 # ChangeLog for app-misc/mplinuxman
-# Copyright 2000-2008 Gentoo Foundation; Distributed under the GPL v2
-# $Header: /var/cvsroot/gentoo-x86/app-misc/mplinuxman/ChangeLog,v 1.15 2008/11/15 13:25:52 flameeyes Exp $
+# Copyright 1999-2009 Gentoo Foundation; Distributed under the GPL v2
+# $Header: /var/cvsroot/gentoo-x86/app-misc/mplinuxman/ChangeLog,v 1.16 2009/07/08 18:50:40 ssuominen Exp $
+
+  08 Jul 2009; Samuli Suominen <ssuominen@gentoo.org> mplinuxman-1.5.ebuild,
+  +files/mplinuxman-1.5-makefile.patch:
+  Use CC and LDFLAGS and fix building with -Wl,--as-needed wrt #277031
 
   15 Nov 2008; Diego E. Pettenò <flameeyes@gentoo.org>
   mplinuxman-1.5.ebuild:
diff --git a/app-misc/mplinuxman/Manifest b/app-misc/mplinuxman/Manifest
index b51c36f0636e..2e51ab581577 100644
--- a/app-misc/mplinuxman/Manifest
+++ b/app-misc/mplinuxman/Manifest
@@ -1,15 +1,6 @@
------BEGIN PGP SIGNED MESSAGE-----
-Hash: SHA1
-
+AUX mplinuxman-1.5-makefile.patch 4744 RMD160 b20578121393b4ce4e8bd5bdde9c679adbd65a70 SHA1 34cda1827f54b68963257783ef060361670bb42e SHA256 23016d4986bf4aa71d4f93098015ded1388e4d18121315a9e14aa0ddab2498df
 AUX mplinuxman-1.5-stringh.patch 1669 RMD160 80b874e980a33ff5d7c54dcc2f6dc64f17541a26 SHA1 b935ef70c56acd7ebb7670e727de6abf3ae8eda9 SHA256 b3974a6f4a6af0230ff5c710048c4393012195b9915eddea5d87d8f22c471c07
 DIST mplinuxman-source-1.5.tar.gz 95331 RMD160 5eb03482810b8b91571d75d07638afde7ae95fe3 SHA1 ea31985978846f2549eb974d37088cf5ad8ed72b SHA256 3c1a5a506feca4cb12f39886361421859642bdbc8c9b4eaf40a321bd8a6e5086
-EBUILD mplinuxman-1.5.ebuild 1377 RMD160 118a22535f7c48cf8047ffb8dfb1d4f069f63900 SHA1 e80f985f5fb0be04f27578983e3ba38b745d9b7e SHA256 bf7da93dcce24a69f606329a7a64a35a94060989d9edd2e22266f08acd375216
-MISC ChangeLog 1959 RMD160 19254b67cecfaa065a715ee06217e38f92269a8e SHA1 9349f1cc061996113408cf10d48dbc76f9c127d6 SHA256 c13e172832aa2eb744ca30be021b0c1719a45f049a95b9f9e540a0c052f24cc5
+EBUILD mplinuxman-1.5.ebuild 1167 RMD160 b2b448d1f8c82e8982af3b6d5f81f329764bef4e SHA1 343aa3e34272a1d6198eaacc8f56bb07c9646dd6 SHA256 1d5d8d55436964a3cb4902cb27ad067bfb326eede8d9bdea122a5c7247597d47
+MISC ChangeLog 2148 RMD160 726b8c12f2783d773a856e4a8ff077e3d0f516fc SHA1 f1383f94e83228d99035c49b3ef7aedb110e3ba2 SHA256 5b6a82605371c9c0d44cab5081b6be6810005a640a3b88a3409b16c656bb054e
 MISC metadata.xml 158 RMD160 6842e2189a50bd8a98e84802c38180ac1421c00e SHA1 703cea5a2109d41f7c87993c1f01d418a4c85174 SHA256 dfb5b47e6836db39fb187301dfcff1c2605e91d13d21db160806a563d8c75f9b
------BEGIN PGP SIGNATURE-----
-Version: GnuPG v2.0.9 (GNU/Linux)
-
-iEYEARECAAYFAkkezeMACgkQAiZjviIA2XgF7QCg4wPo8BgFaNZ6AscQO523qJVb
-a2oAn3JPEesk8rUXG9JByaKla85UBkyQ
-=0kN8
------END PGP SIGNATURE-----
diff --git a/app-misc/mplinuxman/files/mplinuxman-1.5-makefile.patch b/app-misc/mplinuxman/files/mplinuxman-1.5-makefile.patch
new file mode 100644
index 000000000000..9af10e7521f4
--- /dev/null
+++ b/app-misc/mplinuxman/files/mplinuxman-1.5-makefile.patch
@@ -0,0 +1,103 @@
+- Fix translation installation with GNU DESTDIR.
+- Fix building with LDFLAGS="-Wl,--as-needed".
+- Use CC, CFLAGS, LDFLAGS and CPPFLAGS from environment.
+
+diff -ur mplinuxman.orig/extra/mp_util/makefile mplinuxman/extra/mp_util/makefile
+--- mplinuxman.orig/extra/mp_util/makefile	2003-05-22 00:34:41.000000000 +0300
++++ mplinuxman/extra/mp_util/makefile	2009-07-08 21:44:50.000000000 +0300
+@@ -1,5 +1,7 @@
++CC ?= gcc
++
+ mputil: mputil.c ../../f60lib/f60lib.c ../../mp3util.c
+-	gcc -I../../ -I../../f60lib -I../../libusb mputil.c ../../f60lib/f60lib.c ../../mp3util.c ../../libusb/linux.c ../../libusb/descriptors.c ../../libusb/usb.c ../../libusb/error.c -o mputil
++	$(CC) $(LDFLAGS) $(CFLAGS) -I../../ -I../../f60lib -I../../libusb mputil.c ../../f60lib/f60lib.c ../../mp3util.c ../../libusb/linux.c ../../libusb/descriptors.c ../../libusb/usb.c ../../libusb/error.c -o mputil
+ 	ln -s mputil mputil_smart
+ 
+ clean:
+diff -ur mplinuxman.orig/makefile mplinuxman/makefile
+--- mplinuxman.orig/makefile	2006-12-21 17:13:30.000000000 +0200
++++ mplinuxman/makefile	2009-07-08 21:51:46.000000000 +0300
+@@ -1,40 +1,40 @@
+ INCLUDE = `pkg-config --cflags gthread-2.0` `pkg-config --cflags gtk+-2.0` -Ilibusb
+ LIBS = `pkg-config --libs gthread-2.0` `pkg-config --libs gtk+-2.0`
+ export LOCALE_PACKAGE=mplinuxman
+-export LOCALE_DIR=/usr/local/share/locale
++export LOCALE_DIR=$(DESTDIR)/usr/share/locale
+ # remove -D NLS=1 if you want to disable international language support (gettext)
+-COMPILERFLAGS = -O -D MP_VERSION='"1.5"' -D PACKAGE='"$(LOCALE_PACKAGE)"' -D LOCALE_DIR='"$(LOCALE_DIR)"' -D NLS="1"
+-CC = gcc
+-CFLAGS = $(COMPILERFLAGS) $(INCLUDE)
++CPPFLAGS += -D MP_VERSION='"1.5"' -D PACKAGE='"$(LOCALE_PACKAGE)"' -D LOCALE_DIR='"$(LOCALE_DIR)"' -D NLS="1"
++CC ?= gcc
++CFLAGS += $(INCLUDE)
+ 
+ HEADERFILES = dirdialog.h mplinux.h useful.h mputil3.h 
+ OBJECTFILES = dirdialog.o main.o mainwindow.o useful.o mp3util.o playlist.o waitdialog.o mplist.o miscwindow.o 
+ 
+ mplinuxman : $(OBJECTFILES) f60lib.o ini.o  usb.o error.o descriptors.o linux.o
+-	$(CC) -o mplinuxman $(CFLAGS) $(LIBS) $(OBJECTFILES) f60lib.o ini.o  usb.o error.o descriptors.o linux.o ; echo -e "\nDone\nTo install mplinuxman use (this requires root privileges): make install\n" 
++	$(CC) $(LDFLAGS) $(CFLAGS) -o mplinuxman $(OBJECTFILES) f60lib.o ini.o  usb.o error.o descriptors.o linux.o $(LIBS) ; echo -e "\nDone\nTo install mplinuxman use (this requires root privileges): make install\n" 
+ 
+ 
+ f60lib.o : f60lib/f60lib.c
+-	$(CC) -c $(CFLAGS) f60lib/f60lib.c
++	$(CC) -c $(CFLAGS) $(CPPFLAGS) f60lib/f60lib.c
+ 
+ ini.o	: ini/ini.c
+-	$(CC) -c $(CFLAGS) ini/ini.c
++	$(CC) -c $(CFLAGS) $(CPPFLAGS) ini/ini.c
+ 
+ linux.o : libusb/linux.c
+-	$(CC) -c $(CFLAGS) libusb/linux.c
++	$(CC) -c $(CFLAGS) $(CPPFLAGS) libusb/linux.c
+ 
+ descriptors.o : libusb/descriptors.c
+-	$(CC) -c $(CFLAGS) libusb/descriptors.c
++	$(CC) -c $(CFLAGS) $(CPPFLAGS) libusb/descriptors.c
+ 
+ error.o : libusb/error.c
+-	$(CC) -c $(CFLAGS) libusb/error.c
++	$(CC) -c $(CFLAGS) $(CPPFLAGS) libusb/error.c
+ 
+ usb.o : libusb/usb.c
+-	$(CC) -c $(CFLAGS) libusb/usb.c
++	$(CC) -c $(CFLAGS) $(CPPFLAGS) libusb/usb.c
+ 
+ 
+ $(OBJECTFILES) :  %.o: %.c
+-	$(CC) -c $(CFLAGS)  $<
++	$(CC) -c $(CFLAGS) $(CPPFLAGS) $<
+ 
+ clean:
+ 	rm mplinuxman *.o *~ ; exit 0
+@@ -64,11 +64,11 @@
+ 	cp logo.xpm /usr/local/share/pixmaps/mplinuxman.xpm
+ 	mkdir -p /usr/local/share/applications/
+ 	cp mplinuxman.desktop /usr/local/share/applications
+-	mkdir -p /usr/local/share/locale/de/LC_MESSAGES/
+-	mkdir -p /usr/local/share/locale/es/LC_MESSAGES/
+-	mkdir -p /usr/local/share/locale/fr/LC_MESSAGES/
+-	mkdir -p /usr/local/share/locale/ja/LC_MESSAGES/
+-	mkdir -p /usr/local/share/locale/nl/LC_MESSAGES/
++	mkdir -p $(DESTDIR)/usr/share/locale/de/LC_MESSAGES/
++	mkdir -p $(DESTDIR)/usr/share/locale/es/LC_MESSAGES/
++	mkdir -p $(DESTDIR)/usr/share/locale/fr/LC_MESSAGES/
++	mkdir -p $(DESTDIR)/usr/share/locale/ja/LC_MESSAGES/
++	mkdir -p $(DESTDIR)/usr/share/locale/nl/LC_MESSAGES/
+ 	sh  po/install.sh
+ 	
+ uninstall:
+@@ -76,8 +76,8 @@
+ 	rm /usr/local/bin/mputil
+ 	rm /usr/local/share/pixmaps/mplinuxman.xpm
+ 	rm /usr/local/share/applications/mplinuxman.desktop
+-	rm /usr/local/share/locale/de/LC_MESSAGES/mplinuxman.mo
+-	rm /usr/local/share/locale/es/LC_MESSAGES/mplinuxman.mo
+-	rm /usr/local/share/locale/fr/LC_MESSAGES/mplinuxman.mo
+-	rm /usr/local/share/locale/ja/LC_MESSAGES/mplinuxman.mo
+-	rm /usr/local/share/locale/nl/LC_MESSAGES/mplinuxman.mo
++	rm $(DESTDIR)/usr/share/locale/de/LC_MESSAGES/mplinuxman.mo
++	rm $(DESTDIR)/usr/share/locale/es/LC_MESSAGES/mplinuxman.mo
++	rm $(DESTDIR)/usr/share/locale/fr/LC_MESSAGES/mplinuxman.mo
++	rm $(DESTDIR)/usr/share/locale/ja/LC_MESSAGES/mplinuxman.mo
++	rm $(DESTDIR)/usr/share/locale/nl/LC_MESSAGES/mplinuxman.mo
diff --git a/app-misc/mplinuxman/mplinuxman-1.5.ebuild b/app-misc/mplinuxman/mplinuxman-1.5.ebuild
index 6927a0b11cb4..0af61928ec2d 100644
--- a/app-misc/mplinuxman/mplinuxman-1.5.ebuild
+++ b/app-misc/mplinuxman/mplinuxman-1.5.ebuild
@@ -1,7 +1,8 @@
-# Copyright 1999-2008 Gentoo Foundation
+# Copyright 1999-2009 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/app-misc/mplinuxman/mplinuxman-1.5.ebuild,v 1.6 2008/11/15 13:25:52 flameeyes Exp $
+# $Header: /var/cvsroot/gentoo-x86/app-misc/mplinuxman/mplinuxman-1.5.ebuild,v 1.7 2009/07/08 18:50:40 ssuominen Exp $
 
+EAPI=2
 inherit eutils toolchain-funcs
 
 DESCRIPTION="Manager for MPMan F60/55/50 MP3 players."
@@ -13,7 +14,7 @@ SLOT="0"
 KEYWORDS="amd64 ~ppc x86"
 IUSE=""
 
-RDEPEND=">=x11-libs/gtk+-2
+RDEPEND="x11-libs/gtk+:2
 	virtual/mpg123"
 DEPEND="${RDEPEND}
 	dev-util/pkgconfig
@@ -21,22 +22,16 @@ DEPEND="${RDEPEND}
 
 S=${WORKDIR}/${PN}
 
-# This is ugly and someone should write upstream a patch,
-# but it's not me. - drac
-src_unpack() {
-	unpack ${A}
-	cd "${S}"
-	epatch "${FILESDIR}"/${P}-stringh.patch
-
-	sed -e "s:CFLAGS = :CFLAGS = ${CFLAGS} :" \
-		-e 's:/usr/local/share/locale:$(DESTDIR)/usr/share/locale:' \
-		-i makefile
+src_prepare() {
+	epatch "${FILESDIR}"/${P}-stringh.patch \
+		"${FILESDIR}"/${P}-makefile.patch
 }
 
 src_compile() {
-	emake CC="$(tc-getCC)" || die "emake failed."
+	tc-export CC
+	emake || die "emake failed"
 	cd extra/mp_util
-	emake CC="$(tc-getCC)" || die "emake failed."
+	emake || die "emake failed"
 }
 
 src_install() {
@@ -44,10 +39,10 @@ src_install() {
 
 	dodir /usr/share/locale/{de,es,fr,ja,nl}/LC_MESSAGES
 
-	DESTDIR="${D}" emake install-po || die "emake insall-po failed"
+	DESTDIR="${D}" emake install-po || die "emake install-po failed"
 
 	newicon logo.xpm ${PN}.xpm
-	make_desktop_entry ${PN} ${PN} ${PN} "AudioVideo;Audio;GTK;"
+	make_desktop_entry ${PN} ${PN} ${PN} "AudioVideo;Audio;GTK"
 
 	dodoc CHANGES README extra/mp_util/USAGE.txt
 }