Build failed in Jenkins: cyrus-imapd-master #542

[Jeroen van Meeuwen (Kolab Systems)]

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