On Thu, Mar 13, 2014 at 05:46:35PM +0000, Philip Martin wrote:
> Stefan Sperling <s...@apache.org> writes:
> 
> > On Thu, Mar 13, 2014 at 04:16:17PM -0000, s...@apache.org wrote:
> >> Author: stsp
> >> Date: Thu Mar 13 16:16:16 2014
> >> New Revision: 1577223
> 
> >> -    AC_SEARCH_LIBS(bindtextdomain, [intl], [],
> >> +    AC_SEARCH_LIBS(bindtextdomain, [intl],
> >> +                   [
> >> +                    SVN_INTL_LIBS="-lintl"
> 
> That's not correct. AC_SEARCH_LIBS will start with no libs and will only
> go on to check libintl if it doesn't find the symbol.  At this point we
> don't know whether libintl was needed or whether it worked without.

Ah, I see. So we'll probably need to set LIBS to an empty string,
and then check if it was set to anything by AC_SEARCH_LIBS?

> I'm still not really sure why we are trying to set SVN_INTL_LIBS at
> all.  Why is LIBS not enough?

Because otherwise it requires a special hack during pkg-config
file generation. See r1577226 which removed the hack.

Reply via email to