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

Reply via email to