Date: Tue, 15 Sep 2015 21:24:42 +0000 (UTC) From: Dmitry Marakasov <amdmi3@FreeBSD.org> To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r397026 - head/Mk/Uses Message-ID: <201509152124.t8FLOgeN003164@repo.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: amdmi3 Date: Tue Sep 15 21:24:42 2015 New Revision: 397026 URL: https://svnweb.freebsd.org/changeset/ports/397026 Log: - Use -isystem instead of -I to add ${LOCALBASE}/include to the list of header search paths First, this prevents ports which use strict -Werror settings from build failures in third party headers (real life example is devel/pructl which uses -Werror -Weverything and fails in libedit header on actually harmless padding warning). Second, this prevents ports which install headers from picking up their own headers from LOCALBASE instead of using ones from WRKDIR, which leads to any kind of problems when upgrading via ports. PR: 203101 Approved by: bapt Differential Revision: D3618 Modified: head/Mk/Uses/localbase.mk Modified: head/Mk/Uses/localbase.mk ============================================================================== --- head/Mk/Uses/localbase.mk Tue Sep 15 20:55:30 2015 (r397025) +++ head/Mk/Uses/localbase.mk Tue Sep 15 21:24:42 2015 (r397026) @@ -11,9 +11,9 @@ .if !defined(_INCLUDE_USES_LOCALBASE_MK) _INCLUDE_USES_LOCALBASE_MK= yes -CPPFLAGS+= -I${LOCALBASE}/include -CFLAGS+= -I${LOCALBASE}/include -CXXFLAGS+= -I${LOCALBASE}/include +CPPFLAGS+= -isystem ${LOCALBASE}/include +CFLAGS+= -isystem ${LOCALBASE}/include +CXXFLAGS+= -isystem ${LOCALBASE}/include LDFLAGS+= -L${LOCALBASE}/lib # Use CONFIGURE_ENV instead of CMAKE_ARGS because devel/cmake itself also needs
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201509152124.t8FLOgeN003164>