svn commit: samba r18900 - in branches/SAMBA_3_0/source/lib/ldb/common: .

tridge at samba.org tridge at samba.org
Wed Sep 27 01:29:27 GMT 2006


Volker,

 > Next attempt to fix the Solaris build. Not sure about whether to merge this
 > one. Tridge? Metze?

If you merge it, you'd need to expand lib/ldb/ldap.m4 to define
HAVE_LDAP_INITIALIZE. Otherwise this will just disable ldap in ldb :-)

Better still, you could use the lib/ldb/ldap.m4 configure test in
Samba3. It does already test for ldap_initialize(), and doesn't define
HAVE_LDAP unless its available. That keeps the testing for ldap in one
logical place, and means we don't have to put tests for both HAVE_LDAP
and HAVE_LDAP_INITIALIZE everywhere in the code.

Or you could expand the Samba3 configure tests to do the same (perhaps
grabbing bits of ldap.m4 from ldb).

Cheers, Tridge


More information about the samba-cvs mailing list