devel/libssh: don't zero out configure.cppflags, use -isystem.

Zeroing configure.cppflags leads to build failures on older systems (at least 10.4.) Keep the directory in the list, but forcefully move it to the end without beating the build system into submission. Fixes: https://trac.macports.org/ticket/53549
macports · Feb 11, 2017 · daeceb0 · daeceb0
1 parent e6a4196
commit daeceb0
Showing 1 changed file with 6 additions and 4 deletions.
diff --git a/devel/libssh/Portfile b/devel/libssh/Portfile
@@ -41,10 +41,12 @@ post-extract {
 
 cmake.out_of_source yes
 
-# clear configure.cppflags to prevent the cmake portgroup from adding them to
-# cflags, which breaks the build in presence of an older version, because it
-# puts -I$prefix/include before the local include directories of the port
-configure.cppflags
+# -isystem has the added benefit of moving the include
+# directory specified to the end of the include path list.
+# This will help the build system respect custom include
+# paths correctly (i.e., searching them before the MP include
+# directory.)
+configure.cppflags-replace  -I${prefix}/include -isystem${prefix}/include
 configure.args-append \
                     -DWITH_GCRYPT=OFF \
                     -DWITH_INTERNAL_DOC=OFF \