> But then, since what the code is actually trying to get is the include path and not the full cflags, perhaps it could instead request --variable=includedir from pkg-config, and not need to munge the string at all.

Do you mind trying that? I'm sure a patch that does away with the regex filter entirely would be welcome.

The below patch works for me:
diff --git a/build/ac-macros/serf.m4 b/build/ac-macros/serf.m4
index ae11e75..6549147 100644
--- a/build/ac-macros/serf.m4
+++ b/build/ac-macros/serf.m4
@@ -143,7 +143,7 @@ AC_DEFUN(SVN_SERF_PKG_CONFIG,
if $PKG_CONFIG $serf_major --atleast-version=$serf_check_version; then
           AC_MSG_RESULT([yes])
           serf_found=yes
- SVN_SERF_INCLUDES=[`$PKG_CONFIG $serf_major --cflags | $SED -e 's/-D[^ ]*//g'`] + SVN_SERF_INCLUDES="-I`$PKG_CONFIG $serf_major --variable=includedir`"
           SVN_SERF_LIBS=`$PKG_CONFIG $serf_major --libs`
           break
         else


Cheers,
Bruce

Reply via email to