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
> 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
> $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
> conditional is checking for a variable which is never set. But I'll
> 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!