On 2012-04-27 8:32, Greg Banks wrote:
So it turns out there were several things wrong with the code in
configure.ac which checked for LDAP. Most of these have been broken
for
a very long time.
- we check for ldap_initialize() twice, pointlessly
- if given --with-ldap i.e. with no argument, configure checks for a
directory called "yes" and then generates -I/usr/local/include for
LDAP_CPPFLAGS instead of an empty value
- if the LDAP libraries are not detected, we get spurious non-empty
values of LDAP_CPPFLAGS etc
- if LDAP libraries are detected, the final Makefile gets a spurious
extra copy of LDFLAGS
- the recently added automake conditional never worked, as it tests
for
$HAVE_LDAP which is never set, because the AC_CHECK_LIB has a
non-default action-if-set argument, so we compile against the LDAP
headers but don't link against the libraries.
So I fixed all those. Maybe *this* time it'll build...
The AFSKRB checks are probably similarly broken; certainly the
automake
conditional is checking for a variable which is never set. But I'll
let
someone who cares fix that.
Thanks Greg, for this fix - I had run into these a couple of times as
well but had failed in my attempts to fix it. Seems you did it!
Kind regards,
Jeroen van Meeuwen
--
Systems Architect, Kolab Systems AG
e: vanmeeuwen at kolabsys.com
m: +44 74 2516 3817
w: http://www.kolabsys.com
pgp: 9342 BF08
