[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Bug-wget] [bug-wget] Libpsl for cookie domain checking in Wget

From: Giuseppe Scrivano
Subject: Re: [Bug-wget] [bug-wget] Libpsl for cookie domain checking in Wget
Date: Thu, 12 Jun 2014 13:24:02 +0200
User-agent: Gnus/5.13 (Gnus v5.13) Emacs/24.3 (gnu/linux)

Darshit Shah <address@hidden> writes:

> On Wed, Jun 11, 2014 at 5:20 PM, Tim Rühsen <address@hidden> wrote:
>> Am Freitag, 6. Juni 2014, 13:39:32 schrieb Darshit Shah:
>>> I'm facing an issue with the patch I submitted for libpsl and would be
>>> glad if someone could help me.
>>> The configure.ac file does not work as expected. When libpsl is not
>>> installed on a system, the LDFLAGS does not contain -lpsl flag, but
>>> the configure summary shows LIBPSL: Yes.
>>> There is some discrepency in the output that I'd like to fix. The
>>> build completes successfully because the HAVE_LIBPSL variable isn't
>>> set, and Wget compiles without libpsl support. This should however
>>> happen only when --without-libpsl was explicitly specified as a
>>> configure option.
>> This should do it:
>> AC_ARG_WITH(libpsl,
>>     AS_HELP_STRING([--without-libpsl], [disable support for libpsl cookie 
>> checking.]),
>>     [
>>       with_libpsl=no
>>     ], [
>>       AC_CHECK_LIB(psl, psl_builtin,
>>                    [with_libpsl=yes; AC_DEFINE([WITH_LIBPSL], [1], [PSL 
>> support enabled]) LIBS="${LIBS} -lpsl"],
>>                    [with_libpsl=no; AC_MSG_WARN(*** libpsl was not found. 
>> Fallback to Wget builtin cookie checking.)])
>>     ])

I've not tested it but I think this version should fix the problem we
had before.  Just one observation, can we use a different name instead
of "with_libpsl"?  AFAICS, it is used only to display a message at the
end of the configure script, so I would prefer we don't mess with
variables set already by autoconf.
We will then need to set it to no before the AC_CHECK_LIB is used.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]