Re: svn commit: r125933 - /apr/apr-util/trunk/build/apu-conf.m4 /apr/apr-util/trunk/include/apr_ldap.h.in /apr/apr-util/trunk/include/apr_ldap.hnw /apr/apr-util/trunk/include/apr_ldap.hw /apr/apr-util/trunk/ldap/apr_ldap_option.c

[Graham Leggett <mi...@sharp.fm>]

Joe Orton said:

> APR_HAS_LDAPSSL_INSTALL_ROUTINES does not look like it needs to be
> exported via apr_ldap.h, so it should just be flagged in apu_config.h
> using:
>
>  AC_DEFINE(HAVE_LDAPSSL_INSTALL_ROUTINES, 1, [Defined if...])
>
> ...nothing that can be kept private should be exported in the API,
> especially not in a 1.x.y branch.

This would be true if it was assumed that the end user would never want to
use the LDAP native functions themselves, and would always want to use
APR's portability functions - this is a separate debate. But if this was
true, then all the ldap*ssl*() functions need to be removed from this
macro, not just ldapssl_install_routines(), and that can only happen in
APR v2.0 as it's an API change.

An added complication is that the test for the presence of
ldapssl_install_routines() and friends can only happen after a lengthy and
non trivial search for the "right" combination of LDAP libraries on each
platform. As long as this isn't a problem, AC_DEFINE can be used.

Regards,
Graham
--