On 21.05.2015 15:54, Bruce Bye wrote:
>
>> > 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
Um. What happens if we need more than one include path for Serf?
-- Brane
Received on 2015-05-21 17:33:48 CEST