Avoid the dirty sed hack, allow to disable nasm on non-x86, respect CC, CFLAGS and LDFLAGS.

(Portage version: 2.1_pre5-r4)

3 files changed, 39 insertions, 18 deletions

diff --git a/media-video/effectv/ChangeLog b/media-video/effectv/ChangeLog
index 516230411a40..ea405eefcb5d 100644
--- a/media-video/effectv/ChangeLog
+++ b/media-video/effectv/ChangeLog
@@ -1,6 +1,11 @@
 # ChangeLog for media-video/effectv
-# Copyright 1999-2005 Gentoo Foundation; Distributed under the GPL v2
-# $Header: /var/cvsroot/gentoo-x86/media-video/effectv/ChangeLog,v 1.1 2005/03/29 22:50:57 chriswhite Exp $
+# Copyright 1999-2006 Gentoo Foundation; Distributed under the GPL v2
+# $Header: /var/cvsroot/gentoo-x86/media-video/effectv/ChangeLog,v 1.2 2006/03/07 16:34:24 flameeyes Exp $
+
+  07 Mar 2006; Diego Pettenò <flameeyes@gentoo.org>
+  +files/effectv-0.3.10-makefile.patch, effectv-0.3.10.ebuild:
+  Avoid the dirty sed hack, allow to disable nasm on non-x86, respect CC,
+  CFLAGS and LDFLAGS.
 
 *effectv-0.3.10 (30 Mar 2005)
 
diff --git a/media-video/effectv/effectv-0.3.10.ebuild b/media-video/effectv/effectv-0.3.10.ebuild
index ea93ab46994a..a552969ce5e9 100644
--- a/media-video/effectv/effectv-0.3.10.ebuild
+++ b/media-video/effectv/effectv-0.3.10.ebuild
@@ -1,8 +1,8 @@
-# Copyright 1999-2005 Gentoo Foundation
+# Copyright 1999-2006 Gentoo Foundation
 # Distributed under the terms of the GNU General Public License v2
-# $Header: /var/cvsroot/gentoo-x86/media-video/effectv/effectv-0.3.10.ebuild,v 1.1 2005/03/29 22:50:57 chriswhite Exp $
+# $Header: /var/cvsroot/gentoo-x86/media-video/effectv/effectv-0.3.10.ebuild,v 1.2 2006/03/07 16:34:24 flameeyes Exp $
 
-inherit eutils
+inherit eutils toolchain-funcs
 
 DESCRIPTION="EffecTV is a real-time video effect-processor"
 HOMEPAGE="http://effectv.sourceforge.net/"
@@ -11,27 +11,24 @@ LICENSE="GPL-2"
 SLOT="0"
 KEYWORDS="~x86"
 IUSE="mmx"
-DEPEND="virtual/libc
-		dev-lang/nasm
+DEPEND="x86? ( dev-lang/nasm )
 		media-libs/libsdl"
 
 src_unpack() {
 	unpack ${A}
 	cd ${S}
-	# modify path and comment out presets
-	sed -i -e 's:/usr/local:/usr:' \
-		-e 's:^USE_VLOOPBACK:#USE_VLOOPBACK:' \
-		-e 's:^ARCH:#ARCH:' config.mk
-	if ! use mmx; then
-		sed -i -e 's:^USE_MMX:#USE_MMX:' config.mk
-	fi
-	# use Gentoo CFLAGS and compiler
-	sed -i -e "s:\$(CONFIG.arch) \$(CFLAGS.opt):${CFLAGS}:" \
-		-e "s:^CC:#CC:" Makefile
+	epatch "${FILESDIR}/${P}-makefile.patch"
 }
 
 src_compile() {
-	emake || die "emake failed"
+	local mmx
+	local nasm
+
+	use mmx && mmx="yes" || mmx="no"
+	[[ $(tc-arch) == "x86" ]] && nasm="yes" || nasm="no"
+
+	emake CC="$(tc-getCC)" PREFIX="/usr" CFLAGS.opt="${CFLAGS}" \
+		USE_MMX=${mmx} USE_NASM=${nasm} ARCH= || die "emake failed"
 }
 
 src_install() {
diff --git a/media-video/effectv/files/effectv-0.3.10-makefile.patch b/media-video/effectv/files/effectv-0.3.10-makefile.patch
new file mode 100644
index 000000000000..33504129050a
--- /dev/null
+++ b/media-video/effectv/files/effectv-0.3.10-makefile.patch
@@ -0,0 +1,19 @@
+Index: effectv-0.3.10/Makefile
+===================================================================
+--- effectv-0.3.10.orig/Makefile
++++ effectv-0.3.10/Makefile
+@@ -40,12 +40,12 @@ all-recursive:
+ all-am: $(PROGRAM)
+ 
+ $(PROGRAM): $(OBJS) $(LIBEFFECTS) v4lutils/libv4lutils.a
+-	$(CC) -o $@ $(OBJS) $(LIBEFFECTS) $(LIBS)
++	$(CC) $(LDFLAGS) -o $@ $(OBJS) $(LIBEFFECTS) $(LIBS)
+ 
+ $(OBJS): EffecTV.h screen.h video.h palette.h frequencies.h vloopback.h utils.h
+ 
+ install: all-am
+-	$(INSTALL) -s $(PROGRAM) $(bindir)/
++	$(INSTALL) $(PROGRAM) $(bindir)/
+ 	$(INSTALL) effectv.1 $(mandir)/man1/
+ 
+ clean: