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 LDFlags: 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 <[email protected]> wrote: > Darshit Shah <[email protected]> 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
