[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: Darshit Shah
Subject: Re: [Bug-wget] [bug-wget] Libpsl for cookie domain checking in Wget
Date: Sun, 8 Jun 2014 00:24:02 +0530

Hi Giuseppe,

Thanks for the patch. But no, it doesn't solve the problem yes.

Here is the configure summary I get:
configure: Summary of build options:

  Version:           1.15.28-854eb-dirty
  Host OS:           linux-gnu
  Install prefix:    /usr/local
  Compiler:          clang
  CFlags:            -g -O0
  Libs:              -lssl -lcrypto -ldl -lz  -lidn -luuid -lpcre
  SSL:               openssl
  Zlib:              yes
  PSL:               yes
  Digest:            yes
  NTLM:              yes
  OPIE:              yes
  Debugging:         yes

As you can ee, there is a discrepency between the Libs being used and
the detection of PSL. Despite the fact that libpsl is not installed on
my system, configure shows yes.

However, when trying to run make, the relevant variables aren't set
and the `#ifdef HAVE_LIBPSL` statement evaluates to false.

On Fri, Jun 6, 2014 at 2:39 PM, Giuseppe Scrivano <address@hidden> wrote:
> Darshit Shah <address@hidden> writes:
>> 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.
> does something like this fix the problem for you?  I see other libraries
> are using the same pattern, so we should probably fix them as well.
> diff --git a/configure.ac b/configure.ac
> index d2c2e70..9d4d3f8 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -242,8 +242,7 @@ dnl Checks for libraries.
>  dnl
>  AS_IF([test x"$with_libpsl" != xno], [
> -  with_libpsl=yes
> -  AC_CHECK_LIB([psl], [psl_builtin])
> +  AC_CHECK_LIB([psl], [psl_builtin], [with_libpsl=yes])
>  ])
> Also, I don't think we should mess with "with_.*" variables and
> overwrite them, better define something new for the output message.
> Giuseppe

Thanking You,
Darshit Shah

reply via email to

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