On Fri, Jun 29, 2001 at 12:12:57AM -0000, joe@tigris.org wrote:
>   User: joe     
>   Date: 01/06/28 17:12:57
> 
>   Modified:    .        configure.in
>   Log:
>   * configure.in: Fix ra_dav when --disable-shared isn't given (the
>   default).  neon now defaults to not building shared libraries, so force
>   it to do whatever we do.
>...
>   @@ -109,6 +96,25 @@
>    dnl Check for libtool -- we'll definitely need it for all our shared libs!
>    echo "configuring libtool now"
>    AC_PROG_LIBTOOL
>   +
>   +dnl Configure neon --------------------------
>   +
>   +if test "$enable_subdir_config" = "yes"; then
>   +  # neon (as of 0.15) defaults to not building shared libs, so 
>   +  # tell it to do whatever we're doing.
>   +  if test "$enable_shared" = "yes"; then
>   +     args="--enable-shared"
>   +  else
>   +     args="--disable-shared"
>   +  fi
>   +  APR_SUBDIR_CONFIG(neon, $args --with-expat="$abs_srcdir/expat-lite/libexpat.la")
Note that APR_SUBDIR_CONFIG passes the top-level config args to the
sub-config. So... this check is really about filling in a value (to override
the default) for Neon where no cmdline option was specified.
[ the comments kind of get into this, but don't tell the full story... ]
>   +
>   +  AC_MSG_CHECKING([for any extra libraries neon needs])
>   +  # this is not perfect since it will pick up extra -L flags too,
>   +  # but that shouldn't do any real damage.
>   +  NEON_LIBS=`$SHELL $abs_builddir/neon/neon-config --libs | sed -e "s/-lneon//g"`
>   +  AC_MSG_RESULT([$NEON_LIBS])
>   +fi
This probably should *not* be in the conditional. If the subdir config is
skipped (i.e. it was done previously), then we still need to set NEON_LIBS.
Cheers,
-g
-- 
Greg Stein, http://www.lyra.org/
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Oct 21 14:36:32 2006