apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joe Orton <...@manyfish.co.uk>
Subject [PATCH] apr-util: don't add -I/usr/include to INCLUDES
Date Sun, 03 Feb 2002 13:54:57 GMT
The expat detection was adding -I/usr/include to INCLUDES when a
/usr/include/expat.h is found; this is redundant, can mess up include
path ordering, and bleeding edge gcc's even give warnings for it.

(it looks like the code will also do the same for -L/usr/lib in some
circumstances)

Index: build/apu-conf.m4
===================================================================
RCS file: /home/cvspublic/apr-util/build/apu-conf.m4,v
retrieving revision 1.29
diff -u -r1.29 apu-conf.m4
--- build/apu-conf.m4	19 Jan 2002 05:03:57 -0000	1.29
+++ build/apu-conf.m4	3 Feb 2002 13:46:26 -0000
@@ -546,7 +549,9 @@
 AC_SUBST(APR_XML_SUBDIRS)
 AC_SUBST(APR_XML_DIR)
 
-APR_ADDTO(APRUTIL_INCLUDES, [-I$expat_include_dir])
+if test "${expat_include_dir}" != "/usr/include"; then
+   APR_ADDTO(APRUTIL_INCLUDES, [-I$expat_include_dir])
+fi
 APR_ADDTO(APRUTIL_EXPORT_LIBS, [$expat_libs])
 ])
 


Mime
View raw message