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 --
