cpet/freebsd-ports
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

- Use -isystem instead of -I to add ${LOCALBASE}/include to the list of header search paths

Browse Source 
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
This commit is contained in:
Dmitry Marakasov 2015-09-15 21:24:42 +00:00
parent 10af8cf691
commit ecf8298ab7
Notes: svn2git 2021-03-31 03:12:20 +00:00 
svn path=/head/; revision=397026
1 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
Mk/Uses/localbase.mk
View File

@ -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