You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Stefan Sperling <st...@elego.de> on 2008/06/08 17:15:24 UTC
[PATCH] Specify serf's library dependencies in serf.m4
I could not build Subversion trunk with serf (0.2.0) without
the attached patch. Can people using ra_serf test this to
see if it breaks anything for them? I'd like to commit this.
Thanks,
Stefan
[[[
* build/ac-macros/serf.m4: Pass serf's library dependencies to
AC_CHECK_LIB macro.
]]]
Re: [PATCH] Specify serf's library dependencies in serf.m4
Posted by Stefan Sperling <st...@elego.de>.
On Tue, Jun 10, 2008 at 12:51:16PM +0200, Stefan Sperling wrote:
> On Tue, Jun 10, 2008 at 12:15:12AM -0500, Peter Samuelson wrote:
> >
> > [Stefan Sperling]
> > > - AC_CHECK_LIB(serf-0, serf_context_create,[serf_found="yes"])
> > > + AC_CHECK_LIB(serf-0, serf_context_create,[serf_found="yes"], ,
> > > + -lz `$apr_config --link-ld --libs` `$apu_config --link-ld --libs`)
> >
> > Shouldn't that be: $SVN_APRUTIL_EXPORT_LIBS $SVN_APR_EXPORT_LIBS -lz ?
>
> Yep, that works also and looks much cleaner. Thanks!
>
> I'd still like to know whether this patch breaks anything for
> other people compiling with serf.
Since no one has voiced any concerns during almost 2 weeks of
waiting, I've committed this patch in r31841, hoping that
it really won't break anything.
Stefan
>
> [[[
>
> * build/ac-macros/serf.m4: Pass serf's library dependencies to
> AC_CHECK_LIB macro.
>
> ]]]
>
> Index: build/ac-macros/serf.m4
> ===================================================================
> --- build/ac-macros/serf.m4 (revision 31672)
> +++ build/ac-macros/serf.m4 (working copy)
> @@ -22,7 +22,8 @@ AC_DEFUN(SVN_LIB_SERF,
> AC_CHECK_HEADERS(serf.h,[
> save_ldflags="$LDFLAGS"
> LDFLAGS="$LDFLAGS -L$serf_prefix/lib"
> - AC_CHECK_LIB(serf-0, serf_context_create,[serf_found="yes"])
> + AC_CHECK_LIB(serf-0, serf_context_create,[serf_found="yes"], ,
> + $SVN_APRUTIL_EXPORT_LIBS $SVN_APR_EXPORT_LIBS -lz)
> LDFLAGS="$save_ldflags"])
> CPPFLAGS="$save_cppflags"
> fi
Re: [PATCH] Specify serf's library dependencies in serf.m4
Posted by Stefan Sperling <st...@elego.de>.
On Tue, Jun 10, 2008 at 12:15:12AM -0500, Peter Samuelson wrote:
>
> [Stefan Sperling]
> > - AC_CHECK_LIB(serf-0, serf_context_create,[serf_found="yes"])
> > + AC_CHECK_LIB(serf-0, serf_context_create,[serf_found="yes"], ,
> > + -lz `$apr_config --link-ld --libs` `$apu_config --link-ld --libs`)
>
> Shouldn't that be: $SVN_APRUTIL_EXPORT_LIBS $SVN_APR_EXPORT_LIBS -lz ?
Yep, that works also and looks much cleaner. Thanks!
I'd still like to know whether this patch breaks anything for
other people compiling with serf.
[[[
* build/ac-macros/serf.m4: Pass serf's library dependencies to
AC_CHECK_LIB macro.
]]]
Index: build/ac-macros/serf.m4
===================================================================
--- build/ac-macros/serf.m4 (revision 31672)
+++ build/ac-macros/serf.m4 (working copy)
@@ -22,7 +22,8 @@ AC_DEFUN(SVN_LIB_SERF,
AC_CHECK_HEADERS(serf.h,[
save_ldflags="$LDFLAGS"
LDFLAGS="$LDFLAGS -L$serf_prefix/lib"
- AC_CHECK_LIB(serf-0, serf_context_create,[serf_found="yes"])
+ AC_CHECK_LIB(serf-0, serf_context_create,[serf_found="yes"], ,
+ $SVN_APRUTIL_EXPORT_LIBS $SVN_APR_EXPORT_LIBS -lz)
LDFLAGS="$save_ldflags"])
CPPFLAGS="$save_cppflags"
fi
Re: [PATCH] Specify serf's library dependencies in serf.m4
Posted by Peter Samuelson <pe...@p12n.org>.
[Stefan Sperling]
> - AC_CHECK_LIB(serf-0, serf_context_create,[serf_found="yes"])
> + AC_CHECK_LIB(serf-0, serf_context_create,[serf_found="yes"], ,
> + -lz `$apr_config --link-ld --libs` `$apu_config --link-ld --libs`)
Shouldn't that be: $SVN_APRUTIL_EXPORT_LIBS $SVN_APR_EXPORT_LIBS -lz ?
--
Peter Samuelson | org-tld!p12n!peter | http://p12n.org/