LIBREPLACE_LIBS in s3 (Re: [SCM] Samba Shared Repository - branch master updated)

Björn Jacke bj at SerNet.DE
Sun Nov 1 13:50:10 MST 2009


On 2009-11-01 at 11:18 +0100 Stefan (metze) Metzmacher sent off:
> I think talloc and tdb should not get LIBREPLACE_NETWORK_LIBS
> linked in... they don't include "system/network.h" from libreplace.
> And the standalone builds also doesn't use LIBREPLACE_NETWORK at all.

even if unused, linking in object files that have unresolved symbols is bad
manner. I think we could eliminate the dependency the clean way by splitting up
LIBREPLACE_OBJ so that for example the libreplace network object files are only
linked in when needed. But is it really worth doing that?


> I think it's a bit confusing to have LIBREPLACE_LIBS as the same
> as LIBREPLACE_NETWORK_LIBS.

that's because the LIBREPLACE_NETWORK_LIBS are the only libs currently required
by any part of libreplace. Using LIBREPLACE_LIBS inside samba's Makefle is
straigt forward as long as we just have one single LIBREPLACE_OBJ blob. I think
if the object files would be split up the old LIBREPLACE_NETWORK_LIBS would make
ore sense again.

Cheers
Björn
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: not available
URL: <http://lists.samba.org/pipermail/samba-cvs/attachments/20091101/6dfa83b2/attachment-0001.pgp>


More information about the samba-cvs mailing list