Re: svn commit: r124196 - /apr/apr-util/trunk/include/apr_ldap.hw /apr/apr-util/trunk/ldap/apr_ldap_init.c

["William A. Rowe, Jr." <wr...@rowe-clan.net>]

At 06:25 AM 1/5/2005, minfrin@apache.org wrote:
>Author: minfrin
>Date: Wed Jan  5 04:25:33 2005
>New Revision: 124196
>
>URL: http://svn.apache.org/viewcvs?view=rev&rev=124196
>Log:
>The Microsoft version of the ldap_start_tls_s() function has 5 parameters
>instead of openldap's 3. 
>+/*
>+ * On Windows, ldap_start_tls_s is defined with 5 parameters instead
>+ * of the standard 3.
>+ */
>+#define APR_HAS_LDAP_START_TLS_S5 1

Wouldn't testing APR_HAS_MICROSOFT_LDAPSDK be alot cleaner than
adding more HAS macros?

Bill

Re: svn commit: r124196 - /apr/apr-util/trunk/include/apr_ldap.hw /apr/apr-util/trunk/ldap/apr_ldap_init.c

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

William A. Rowe, Jr. wrote:

> Wouldn't testing APR_HAS_MICROSOFT_LDAPSDK be alot cleaner than
> adding more HAS macros?

I could not find a clean way of getting autoconf to detect the LDAP SDK 
(I am no expert on autoconf or complex #ifdef magic, it made my brain 
hurt). The immediate need was solved by doing string comparisons on the 
only #define that gives away this information: LDAP_VENDOR_NAME. If 
someone can show me how to convince autoconf to actually detect the SDK 
instead of the klunky thing that's there now I would be hugely grateful :)

Regards,
Graham
--