Date: Mon, 7 Jun 2010 11:36:48 +0300 (EEST) From: Andriy Gapon <avg@icyb.net.ua> To: FreeBSD-gnats-submit@freebsd.org Cc: multimedia@freebsd.org Subject: [PATCH] multimedia/mkvtoolnix: use += for setting CPPFLAGS and LDFLAGS Message-ID: <201006070836.o578amrC069748@trant.local.>
next in thread | raw e-mail | index | archive | help
>Submitter-Id: current-users
>Originator: Andriy Gapon
>Organization:
>Confidential: no
>Synopsis: [PATCH] multimedia/mkvtoolnix: use += for setting CPPFLAGS and LDFLAGS
>Severity: non-critical
>Priority: low
>Category: ports
>Class: change-request
>Release: FreeBSD 9.0-CURRENT amd64
>Environment:
>Description:
Right now this port simply sets CPPFLAGS and LDFLAGS thus overriding any local
tuning that a user might want to do via e.g. /etc/make.conf.
I think that in general it is recommended that CPPFLAGS, LDFLAGS, etc are
amended, not overridden, in ports Makefiles.
Port maintainer (multimedia@FreeBSD.org) is cc'd.
Generated with FreeBSD Port Tools 0.99
>How-To-Repeat:
>Fix:
--- mkvtoolnix-3.3.0_2.patch begins here ---
diff -ruN --exclude=CVS /usr/ports/multimedia/mkvtoolnix.orig/Makefile /usr/ports/multimedia/mkvtoolnix/Makefile
--- /usr/ports/multimedia/mkvtoolnix.orig/Makefile 2010-06-06 23:43:58.000000000 +0300
+++ /usr/ports/multimedia/mkvtoolnix/Makefile 2010-06-07 11:33:29.069800315 +0300
@@ -28,8 +28,8 @@
USE_GMAKE= yes
USE_ICONV= yes
GNU_CONFIGURE= yes
-CPPFLAGS= -I${LOCALBASE}/include
-LDFLAGS= -L${LOCALBASE}/lib
+CPPFLAGS+= -I${LOCALBASE}/include
+LDFLAGS+= -L${LOCALBASE}/lib
CONFIGURE_ENV= CPPFLAGS="${CPPFLAGS}" LDFLAGS="${LDFLAGS}" \
ac_cv_path_PO4A=no
CONFIGURE_ARGS= --enable-lzo --enable-bz2 --disable-qt \
--- mkvtoolnix-3.3.0_2.patch ends here ---
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201006070836.o578amrC069748>