QA clean up and fixing build with --as-needed wrt bug 321303. Thanks to Diego for reporting. With beandog's blessing

Package-Manager: portage-2.1.8.3/cvs/Linux x86_64

4 files changed, 136 insertions, 17 deletions

diff --git a/media-tv/ivtv-utils/ChangeLog b/media-tv/ivtv-utils/ChangeLog
index e19ddc2b3631..48a205e4371c 100644
--- a/media-tv/ivtv-utils/ChangeLog
+++ b/media-tv/ivtv-utils/ChangeLog
@@ -1,6 +1,11 @@
 # ChangeLog for media-tv/ivtv-utils
 # Copyright 1999-2010 Gentoo Foundation; Distributed under the GPL v2
-# $Header: /var/cvsroot/gentoo-x86/media-tv/ivtv-utils/ChangeLog,v 1.9 2010/05/21 01:55:17 beandog Exp $
+# $Header: /var/cvsroot/gentoo-x86/media-tv/ivtv-utils/ChangeLog,v 1.10 2010/06/29 19:32:00 xarthisius Exp $
+
+  29 Jun 2010; Kacper Kowalik <xarthisius@gentoo.org>
+  ivtv-utils-1.4.0-r1.ebuild, +files/ivtv-utils-1.4.0-gentoo.patch:
+  QA clean up and fixing build with --as-needed wrt bug 321303. Thanks to
+  Diego for reporting.
 
   21 May 2010; Steve Dibb <beandog@gentoo.org> ivtv-utils-1.4.0-r1.ebuild:
   Add dep on v4l2-ctl
diff --git a/media-tv/ivtv-utils/Manifest b/media-tv/ivtv-utils/Manifest
index ed593f888dbd..f5df82807097 100644
--- a/media-tv/ivtv-utils/Manifest
+++ b/media-tv/ivtv-utils/Manifest
@@ -1,8 +1,21 @@
+-----BEGIN PGP SIGNED MESSAGE-----
+Hash: SHA1
+
 AUX ivtv-utils-1.3.0-2.6.27.patch 5014 RMD160 7a1e3982cab722ac8858ec4989d2e1fd6f3d3081 SHA1 c5e6ca21d3578dfa708ff33f125f6c331fee3185 SHA256 b08e9449c67c279a30e324d435d421ff748109af909bb39dc8549784784ebcfb
+AUX ivtv-utils-1.4.0-gentoo.patch 2470 RMD160 0c0717d5b34757d0a05a57d0a675f1899e4a6d73 SHA1 a58249717f98f45e37a287cd913cd111bf7a10dd SHA256 23a4589e799ff0bf06149aac8f9d616441ef8a7d00b51e900c46a2e8e9758c81
 DIST ivtv-utils-1.3.0.tar.gz 173745 RMD160 b42572277babd840868adb641d8434b180d43315 SHA1 bc4d7003ce9fb75ee137e8f79c880e4fb45d1137 SHA256 66daf16ae2f124e31131451e52295072c2615d9931a95f7eccee58c5351af498
 DIST ivtv-utils-1.4.0.tar.gz 192733 RMD160 841b46c64dc57e04e603a1a074a55be1d6ea5ea3 SHA1 ea046eec2511ad00395e9de5ba693c069190d2c4 SHA256 702c365da723ce419bd49f25f88eb3a028f0594699f9c9710c713eba022ac911
 EBUILD ivtv-utils-1.3.0-r1.ebuild 3464 RMD160 c9b8d4640e1276a290d3f582a9d760b73a747831 SHA1 4bc304dbf684921b12c86215f9c22543ae6edbcd SHA256 982a71066354dbef76d5ae9d1a33f2043f9a596a16fdd5c7b426520f94f6194c
 EBUILD ivtv-utils-1.3.0-r2.ebuild 3011 RMD160 9d2e23aba890a01cb3a0c4ccbf0a0938fe8b6b77 SHA1 d0a9331ad2a943278eae6fab6bd3b46d313516fc SHA256 b1cd7cee5591682f7251f630c1a902d5453835f188dbd2a5b480ce111455a67e
-EBUILD ivtv-utils-1.4.0-r1.ebuild 3101 RMD160 c18af5b87f1bd7ee6b45c4be76f090ea14516aeb SHA1 e29421dfe0c8e69d84b310c3eac4d585e7296dd3 SHA256 0352f4ba4442a917728ee363cff921ed3ef6c18e4fd9fa37b9f6a08353f36d83
-MISC ChangeLog 1886 RMD160 e8e4003f0c5c2ca09b3d70c9684aa45e37d3aabe SHA1 2e642b2dd3b91dbbf2d2f1b6ca2c9647124f2529 SHA256 0bff09c737ecbfc7c8501ef14cb7b13ef105e283f3fd6cb8e9c7da5b1094d680
+EBUILD ivtv-utils-1.4.0-r1.ebuild 3056 RMD160 2d00e051c1d155d1b65b9dfa0ec08993b232136c SHA1 d36e5d08a74324d2deabbfd510b52e1639a93580 SHA256 4033618cbc74d0310b3362bdede493a88445371b245f072f0206d9cebe6208d6
+MISC ChangeLog 2110 RMD160 e09ab2a1c4217f47992a3705fc34ef5133b3653f SHA1 9604e819991eb32048267d290c24e3d623278fcc SHA256 c8fcbd3371ccbecea90c19997c59749afaf733ce2b320381e507ab679e608e68
 MISC metadata.xml 494 RMD160 a8c45c9e2ad783198d04b68c755398d915270499 SHA1 1a04f1e7b3be80d5a875559fddbc5b681ba9254b SHA256 99e7a214b2abd11188552236eb67309ccde04d0a66209ed0fd48589eed258bb4
+-----BEGIN PGP SIGNATURE-----
+Version: GnuPG v2.0.15 (GNU/Linux)
+
+iJwEAQECAAYFAkwqSjoACgkQIiMqcbOVdxQaSAQAi/Ozi/V7ZaOAYmt4bUgekFpc
+pp3zeM4dVYOWBLfnj8a9y0HAJ1uJCB6NwROuo09oP0tCCDPR+Nc6kDsi71kXoL5W
+IQQFnqupv6//prYJKxsK1+XSrg+fbEwZ5QeDg6J0IuBgoT6H8ojvx7UDL2EeDy0F
+ZSMiygOnsGmSE8N05U4=
+=Wzrg
+-----END PGP SIGNATURE-----
diff --git a/media-tv/ivtv-utils/files/ivtv-utils-1.4.0-gentoo.patch b/media-tv/ivtv-utils/files/ivtv-utils-1.4.0-gentoo.patch
new file mode 100644
index 000000000000..512e3c4ff9f0
--- /dev/null
+++ b/media-tv/ivtv-utils/files/ivtv-utils-1.4.0-gentoo.patch
@@ -0,0 +1,99 @@
+Fixing multiple QA issues
+ * respect user {CC,CXX,LD}FLAGS
+ * fix linking order
+ * make job server for install phase
+
+Don't install v4l2-ctl (bug #278255)
+Don't install duplicate kernel headers (bug #244584)
+
+https://bugs.gentoo.org/show_bug.cgi?id=321303
+
+Patch written by Kacper Kowalik <xarthisius.kk@gmail.com>
+
+--- test/Makefile
++++ test/Makefile
+@@ -7,13 +7,13 @@
+ 	ivtv-osd-dma-test ivtv-fb-colormap-test \
+ 	ivtv-fb-16-bit-test ps-analyzer
+ 
+-CFLAGS = -I../utils -D_GNU_SOURCE -O2 -Wall
+-CXXFLAGS = $(CFLAGS)
+-LDFLAGS = -lm
++CFLAGS += -I../utils -D_GNU_SOURCE -Wall
++CXXFLAGS += -I../utils -D_GNU_SOURCE -Wall
++LDLIBS = -lm
+ 
+ all: $(EXES)
+ 
+-install: all
++install:
+
+ clean: 
+ 	rm -f *.o $(EXES)
+--- utils/Makefile.orig	2010-05-24 15:35:11.321677941 +0000
++++ utils/Makefile	2010-05-24 15:35:15.595615306 +0000
+@@ -9,41 +9,39 @@
+ X86_EXES := ivtvplay ivtv-mpegindex #ivtv-encoder
+ endif
+ 
+-EXES := v4l2-ctl ivtv-ctl ivtv-radio $(X86_EXES)
++EXES := ivtv-ctl ivtv-radio $(X86_EXES)
+ 
+ BIN := $(EXES) ivtv-tune/ivtv-tune cx25840ctl/cx25840ctl
+ 
+ 
+ HEADERS := linux/ivtv.h linux/ivtvfb.h
+ 
+-CFLAGS = -D_GNU_SOURCE -O2 -Wall -g -I.
+-CXXFLAGS = $(CFLAGS)
++CFLAGS   += -D_GNU_SOURCE -Wall -I.
++CXXFLAGS += -D_GNU_SOURCE -Wall -I.
+ 
+-all:: $(EXES)
+-	$(MAKE) CFLAGS="$(CFLAGS)" -C ivtv-tune
+-	$(MAKE) CFLAGS="$(CFLAGS)" -C cx25840ctl
++all: $(EXES)
++	$(MAKE) $(MAKEOPTS) CFLAGS="$(CFLAGS)" -C ivtv-tune
++	$(MAKE) $(MAKEOPTS) CFLAGS="$(CFLAGS)" -C cx25840ctl
+ 
+ ivtv-ctl: ivtv-ctl.o
+-	$(CC) -lm -o $@ $^
++	$(CC) $(CFLAGS) $(LDFLAGS) -o $@ $^ -lm
+ 
+ v4l2-ctl: v4l2-ctl.o
+-	$(CXX) -lm -o $@ $^
++	$(CXX) $(CXXFLAGS) $(LDFLAGS) -o $@ $^ -lm
+ 
+ v4l2-dbg: v4l2-dbg.o v4l2-driverids.o v4l2-chipids.o
+-	$(CXX) -lm -o $@ $^
++	$(CXX) $(CXXFLAGS) $(LDFLAGS) -o $@ $^ -lm
+ 
+ ivtvplay: ivtvplay.cc
+-	$(CXX) $(CXXFLAGS) -lm -lpthread -o $@ $^
++	$(CXX) $(CXXFLAGS) $(LDFLAGS) -o $@ $^ -lm -lpthread
+ 
+ encoder.o: encoder.c
+ 	$(CC) $(CFLAGS) -DVIDEO_PORT=0 -D_FILE_OFFSET_BITS=64 -D_LARGEFILE_SOURCE -c $^
+ 
+ ivtv-encoder: enc_mindex.o enc_chann.o encoder.o
+-	$(CC) -lpthread -o $@ $^
++	$(CC) $(CFLAGS) $(LDFLAGS) -o $@ $^ -lpthread
+ 
+-install: all
+-	install -d $(DESTDIR)/$(HDRDIR)
+-	install -m 0644 $(HEADERS) $(DESTDIR)/$(HDRDIR)
++install:
+ 	install -d $(DESTDIR)/$(BINDIR)
+ 	install -m 0755 $(BIN) $(DESTDIR)/$(BINDIR)
+ 
+--- Makefile
++++ Makefile
+@@ -1,6 +1,6 @@
+ all clean install:
+-	make -C utils $@
+-	make -C test $@
++	$(MAKE) -C utils $@
++	$(MAKE) -C test $@
+ 
+ distclean: clean
+ 
diff --git a/media-tv/ivtv-utils/ivtv-utils-1.4.0-r1.ebuild b/media-tv/ivtv-utils/ivtv-utils-1.4.0-r1.ebuild
index 0aedc6e86cc9..fb6f75e04f14 100644
--- a/media-tv/ivtv-utils/ivtv-utils-1.4.0-r1.ebuild
+++ b/media-tv/ivtv-utils/ivtv-utils-1.4.0-r1.ebuild
@@ -1,10 +1,10 @@
 # Copyright 1999-2010 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/media-tv/ivtv-utils/ivtv-utils-1.4.0-r1.ebuild,v 1.3 2010/05/21 01:55:17 beandog Exp $
+# $Header: /var/cvsroot/gentoo-x86/media-tv/ivtv-utils/ivtv-utils-1.4.0-r1.ebuild,v 1.4 2010/06/29 19:32:00 xarthisius Exp $
 
 EAPI=2
 
-inherit eutils linux-mod linux-info
+inherit eutils linux-mod linux-info toolchain-funcs
 
 DESCRIPTION="IVTV utilities for Hauppauge PVR PCI cards"
 HOMEPAGE="http://www.ivtvdriver.org"
@@ -57,20 +57,22 @@ pkg_setup() {
 	BUILD_PARAMS="KDIR=${KV_DIR}"
 }
 
-src_install() {
-	make DESTDIR="${D}" PREFIX="/usr" install || die "failed to install"
-	use perl && dobin utils/perl/*.pl
-
-	# Shouldn't be installing linux headers, bug 273165
-	rm "${D}"/usr/include/linux/ivtv.h
-	rm "${D}"/usr/include/linux/ivtvfb.h
+src_prepare() {
+	epatch "${FILESDIR}"/${P}-gentoo.patch
+}
 
-	# Installed separately now
-	rm "${D}"/usr/bin/v4l2-ctl
+src_compile() {
+	tc-export CC CXX
+	emake || die
+}
 
-	cd "${S}"
-	dodoc README doc/* ChangeLog
-	use perl && dodoc utils/perl/README.ptune
+src_install() {
+	emake DESTDIR="${D}" PREFIX="/usr" install || die "failed to install"
+	dodoc README doc/* ChangeLog || die
+	if use perl; then
+		dobin utils/perl/*.pl || die
+		dodoc utils/perl/README.ptune || die
+	fi
 }
 
 pkg_postinst() {