CONFIGURE_ENV and CPPFLAGS

Eitan Adler lists at eitanadler.com
Fri Feb 22 23:46:13 UTC 2013


Can you please sanity check the following?

commit 687519c34ad3423752268a0a20d079fda9cea8b2
Author: Eitan Adler <lists at eitanadler.com>
Date:   Fri Feb 22 18:43:30 2013 -0500

    Using CONFIGURE_ENV for CPPFLAGS and
    LDFLAGS is not needed since r282433.

    PR:		docs/174827
    Submitted by:	Dominique Goncalves <dominique.goncalves at gmail.com>

diff --git a/en_US.ISO8859-1/books/porters-handbook/book.xml
b/en_US.ISO8859-1/books/porters-handbook/book.xml
index ef50257..c64cdc7 100644
--- a/en_US.ISO8859-1/books/porters-handbook/book.xml
+++ b/en_US.ISO8859-1/books/porters-handbook/book.xml
@@ -5726,17 +5726,13 @@ IGNORE=	may not be redistributed because of
licensing reasons. Please visit <rep
 CPPFLAGS+=	-I${LOCALBASE}/include
 LDFLAGS+=	-L${LOCALBASE}/lib

-GNU_CONFIGURE=	yes
-CONFIGURE_ENV=	CPPFLAGS="${CPPFLAGS}" \
-		LDFLAGS="${LDFLAGS}"</programlisting>
+GNU_CONFIGURE=	yes</programlisting>

 	<para>Of course, the code can be more compact if there are no
 	  more flags to pass to <command>configure</command>:</para>

 	<programlisting>USE_GETTEXT=	yes
-GNU_CONFIGURE=	yes
-CONFIGURE_ENV=	CPPFLAGS="-I${LOCALBASE}/include" \
-		LDFLAGS="-L${LOCALBASE}/lib"</programlisting>
+GNU_CONFIGURE=	yes</programlisting>
       </sect2>

       <sect2>


-- 
Eitan Adler


More information about the freebsd-ports mailing list