Re: [BUG] Configure script fails to detect serf library due to ignored dependencies

[Daniel Shahaf <da...@elego.de>]

Daniel Richard G. wrote on Sun, Aug 25, 2013 at 18:12:05 -0400:
> The problem, however, is that...
> 
>     $ grep ^dep /tmp/apr-build/_install/lib/libapr-1.la
>     dependency_libs=' -lresolv -luuid -lsendfile -lrt -lsocket -lnsl -lpthread'
> 
>     $ grep ^Libs: /tmp/apr-build/_install/lib/pkgconfig/apr-1.pc
>     Libs: -L${libdir} -lapr-${APR_MAJOR_VERSION} -lresolv -luuid -lsendfile -lrt -lsocket -lnsl  -lpthread
> 
> ...this dependency information is already contained in the Libtool and
> pkg-config metadata for libapr, yet Subversion's configure script is
> not using it. Setting LIBS should not be necessary.

So that would appear to be a bug in serf's pkg-config file, no?

Note that serf's build system was completely changed in 1.3.  (You don't
mention which version of libserf you use?)

Re: [BUG] Configure script fails to detect serf library due to ignored dependencies

["Daniel Richard G." <sk...@iSKUNK.ORG>]

On Wed, 2013 Aug 28 6:39+0300, Daniel Shahaf wrote:
>
> So that would appear to be a bug in serf's pkg-config file, no?

No, Serf's doing the right thing:

$ grep ^Libs.private: /tmp/serf-build/_install/lib/pkgconfig/serf-1.pc
Libs.private: -lm   /tmp/apr-util-build/_install/lib/libaprutil-1.la     -lexpat -L/nfs/freeport/arch/sunos_x86_64/lib  -lssl -lcrypto             /tmp/apr-build/_install/lib/libapr-1.la -lresolv -luuid -lsendfile -lrt -lsocket -lnsl -lpthread -lm -lz

$ grep ^dep /tmp/serf-build/_install/lib/libserf-1.la
dependency_libs=' -R/nfs/freeport/arch/sunos_x86_64/lib -L/nfs/freeport/arch/sunos_x86_64/lib -L/tmp/apr-util-build/_install/lib /tmp/apr-util-build/_install/lib/libaprutil-1.la /tmp/apr-util-build/_install/lib/libexpat.la /tmp/apr-build/_install/lib/libapr-1.la -lresolv -luuid -lsendfile -lrt -lsocket -lnsl -lpthread -lm -lz -lssl -lcrypto'

> Note that serf's build system was completely changed in 1.3.  (You
> don't mention which version of libserf you use?)

This is libserf 1.2.1, the version pulled in by Subversion's
get-deps.sh script.


--Daniel


-- 
Daniel Richard G. || skunk@iSKUNK.ORG
My ASCII-art .sig got a bad case of Times New Roman.