Re: unixware and --with-ldap

Dear Tom,

Thanks for you advice.

Here's the diff for configure.in that works for me.
Best regards

---
*** configure.in.orig jeu déc 14 18:35:27 2006
--- configure.in jeu déc 14 18:35:37 2006
***************
*** 1122,1137 ****
if test "$with_ldap" = yes ; then
_LIBS="$LIBS"
if test "$PORTNAME" != "win32"; then
! AC_CHECK_LIB(ldap, ldap_bind, [], [AC_MSG_ERROR([library 'ldap' is required for LDAP])])
! LDAP_LIBS_BE="-lldap"
if test "$enable_thread_safety" = yes; then
# on some platforms ldap_r fails to link without PTHREAD_LIBS
AC_CHECK_LIB(ldap_r, ldap_simple_bind, [],
[AC_MSG_ERROR([library 'ldap_r' is required for LDAP])],
! [$PTHREAD_LIBS])
LDAP_LIBS_FE="-lldap_r"
else
! LDAP_LIBS_FE="-lldap"
fi
else
AC_CHECK_LIB(wldap32, ldap_bind, [], [AC_MSG_ERROR([library 'wldap32' is required for LDAP])])
--- 1122,1139 ----
if test "$with_ldap" = yes ; then
_LIBS="$LIBS"
if test "$PORTNAME" != "win32"; then
! AC_CHECK_LIB(ldap, ldap_bind, [],
! [AC_MSG_ERROR([library 'ldap' is required for LDAP])],
! [$EXTRA_LDAP_LIBS])
! LDAP_LIBS_BE="-lldap $EXTRA_LDAP_LIBS"
if test "$enable_thread_safety" = yes; then
# on some platforms ldap_r fails to link without PTHREAD_LIBS
AC_CHECK_LIB(ldap_r, ldap_simple_bind, [],
[AC_MSG_ERROR([library 'ldap_r' is required for LDAP])],
! [$PTHREAD_LIBS $EXTRA_LDAP_LIBS])
LDAP_LIBS_FE="-lldap_r"
else
! LDAP_LIBS_FE="-lldap $EXTRA_LDAP_LIBS"
fi
else
AC_CHECK_LIB(wldap32, ldap_bind, [], [AC_MSG_ERROR([library 'wldap32' is required for LDAP])])

On Mon, 11 Dec 2006, Tom Lane wrote:

> Date: Mon, 11 Dec 2006 11:26:14 -0500
> From: Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>
> To: Andrew Dunstan <andrew(at)dunslane(dot)net>
> Cc: ohp(at)pyrenet(dot)fr, pgsql-hackers list <pgsql-hackers(at)postgresql(dot)org>
> Subject: Re: [HACKERS] unixware and --with-ldap
>
> Andrew Dunstan <andrew(at)dunslane(dot)net> writes:
> > The right way to do this I think is to put an entry adjusting LIBS in
> > src/makefiles/Makefile.unixware, but first it looks like we need to
> > propagate the with-ldap switch into src/Makefile.global
>
> The Makefile is far too late --- this has to be adjusted at configure
> time, else configure will conclude that libldap isn't available.
> (I assume that Unixware's linker is sane enough that the link test
> will fail without the supporting libraries.)
>
> I'd be inclined to do something like add
>
> EXTRA_LDAP_LIBS="-llber -lresolv"
>
> in src/template/unixware, and then in configure.in add this where
> needed:
>
> if test "$with_ldap" = yes ; then
> _LIBS="$LIBS"
> if test "$PORTNAME" != "win32"; then
> AC_CHECK_LIB(ldap, ldap_bind, [$EXTRA_LDAP_LIBS], [AC_MSG_ERROR([library 'ldap' is required for LDAP])])
> LDAP_LIBS_BE="-lldap $EXTRA_LDAP_LIBS"
> if test "$enable_thread_safety" = yes; then
> # on some platforms ldap_r fails to link without PTHREAD_LIBS
> AC_CHECK_LIB(ldap_r, ldap_simple_bind, [$EXTRA_LDAP_LIBS],
> [AC_MSG_ERROR([library 'ldap_r' is required for LDAP])],
> [$PTHREAD_LIBS])
> LDAP_LIBS_FE="-lldap_r $EXTRA_LDAP_LIBS"
> else
> LDAP_LIBS_FE="-lldap $EXTRA_LDAP_LIBS"
> fi
> else
> AC_CHECK_LIB(wldap32, ldap_bind, [], [AC_MSG_ERROR([library 'wldap32' is required for LDAP])])
> LDAP_LIBS_FE="-lwldap32"
> LDAP_LIBS_BE="-lwldap32"
> fi
> LIBS="$_LIBS"
> fi
>
> I'm not in a position to test that though ...
>
> regards, tom lane
>

--
Olivier PRENANT Tel: +33-5-61-50-97-00 (Work)
15, Chemin des Monges +33-5-61-50-97-01 (Fax)
31190 AUTERIVE +33-6-07-63-80-64 (GSM)
FRANCE Email: ohp(at)pyrenet(dot)fr
------------------------------------------------------------------------------
Make your life a dream, make your dream a reality. (St Exupery)