On Thursday 20 November 2014 15:36:57 Darshit Shah wrote: > On 11/20, Darshit Shah wrote: > >How does this look for another attempt at the configure file? > > Here's another patch that uses pkg-config to check for libpsl > > Also, if there are no objections, I'll also push the patch that adds -Wextra > to the default CFLAGS.
Sorry, you forgot
# correct $LIBPSL_LIBS (in libpsl <= 0.6.0)
AS_IF([test "x$LIBPSL_LIBS" = "x-llibpsl "], [LIBPSL_LIBS="-lpsl"])
Missing this, ./configure will not work any more (you called it spurious
errors).
Please add it ASAP, just can't build Wget right now.
My code example was:
AC_ARG_WITH(libpsl, AS_HELP_STRING([--without-libpsl], [disable support for
libpsl cookie checking]), with_libpsl=$withval, with_libpsl=yes)
AS_IF([test "x$with_libpsl" != xno], [
PKG_CHECK_MODULES([LIBPSL], libpsl, [
with_libpsl=yes
# correct $LIBPSL_LIBS (in libpsl <= 0.6.0)
AS_IF([test "x$LIBPSL_LIBS" = "x-llibpsl "], [LIBPSL_LIBS="-lpsl"])
LIBS="$LIBPSL_LIBS $LIBS"
CFLAGS="$LIBPSL_CFLAGS $CFLAGS"
AC_DEFINE([WITH_LIBPSL], [1], [PSL support enabled])
], [
AC_SEARCH_LIBS(psl_builtin, psl,
[with_libpsl=yes; AC_DEFINE([WITH_LIBPSL], [1], [PSL support enabled])],
[with_libpsl=no; AC_MSG_WARN(*** libpsl was not found. Fallback to
builtin cookie checking.)])
])
])
Tim
signature.asc
Description: This is a digitally signed message part.
