Re: [HACKERS] unixware and --with-ldap

2007-02-02 Thread Bruce Momjian
Clarification, this is the email used to make the patch that was backpatched. --- Albe Laurenz wrote: I have tried --with-thread-safety and configure fails on ldap check because for some reason CTHREAD_FLAGS (-Kpthread

Re: [HACKERS] unixware and --with-ldap

2006-12-18 Thread Albe Laurenz
If libldap_r.so does require the same libs, please add $EXTRA_LDAP_LIBS to the 'LDAP_LIBS_FE=-lldap_r' line as well. Actually, I did that in the patch-as-committed. It seems to me that it's probably harmless even if not needed. I have tried --with-thread-safety and configure fails on ldap

Re: [HACKERS] unixware and --with-ldap

2006-12-18 Thread ohp
Hi Albe, On Mon, 18 Dec 2006, Albe Laurenz wrote: Date: Mon, 18 Dec 2006 09:31:35 +0100 From: Albe Laurenz [EMAIL PROTECTED] To: ohp@pyrenet.fr, Tom Lane [EMAIL PROTECTED] Cc: pgsql-hackers list pgsql-hackers@postgresql.org Subject: RE: [HACKERS] unixware and --with-ldap If libldap_r.so

Re: [HACKERS] unixware and --with-ldap

2006-12-18 Thread Albe Laurenz
I have tried --with-thread-safety and configure fails on ldap check because for some reason CTHREAD_FLAGS (-Kpthread for UW) is missing on compile command, although present before that. I can't find why You mean PTHREAD_FLAGS, right? Nope,I mean PTHREAD_CFLAGS witch is defined in

Re: [HACKERS] unixware and --with-ldap

2006-12-16 Thread ohp
On Fri, 15 Dec 2006, Tom Lane wrote: Date: Fri, 15 Dec 2006 10:20:44 -0500 From: Tom Lane [EMAIL PROTECTED] To: Albe Laurenz [EMAIL PROTECTED] Cc: ohp@pyrenet.fr, pgsql-hackers list pgsql-hackers@postgresql.org Subject: Re: [HACKERS] unixware and --with-ldap Albe Laurenz [EMAIL PROTECTED

Re: [HACKERS] unixware and --with-ldap

2006-12-15 Thread Albe Laurenz
Here's the diff for configure.in that works for me. I have a concern about this patch: 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, [],

Re: [HACKERS] unixware and --with-ldap

2006-12-15 Thread ohp
Hi Albe, Thanks for your remarks On Fri, 15 Dec 2006, Albe Laurenz wrote: Date: Fri, 15 Dec 2006 09:54:13 +0100 From: Albe Laurenz [EMAIL PROTECTED] To: ohp@pyrenet.fr, Tom Lane [EMAIL PROTECTED] Cc: pgsql-hackers list pgsql-hackers@postgresql.org Subject: RE: [HACKERS] unixware

Re: [HACKERS] unixware and --with-ldap

2006-12-15 Thread Albe Laurenz
Olivier PRENANT wrote: You left out the case where --enable_thread_safety is specified. In that case, the frontend has to be linked with libldap_r.so instead of libldap.so. Yes, this was on purpose, my goal is to try to make a second patch when... Does libldap_r.so _not_ require any

Re: [HACKERS] unixware and --with-ldap

2006-12-15 Thread Martijn van Oosterhout
On Fri, Dec 15, 2006 at 01:06:08PM +0100, Albe Laurenz wrote: I guess that adding $EXTRA_LDAP_LIBS to -lldap_r will be enough, judging from the evidence on Linux. Linux doesn't need the extra libraries, it's linked properly. Additionally, on Linux there is no difference between ldap and

Re: [HACKERS] unixware and --with-ldap

2006-12-15 Thread Albe Laurenz
Martijn van Oosterhout wrote: I guess that adding $EXTRA_LDAP_LIBS to -lldap_r will be enough, judging from the evidence on Linux. Linux doesn't need the extra libraries, it's linked properly. Yes - what I mean is, libldap_r.so references only NEEDED libsasl.so.7 NEEDED

Re: [HACKERS] unixware and --with-ldap

2006-12-15 Thread Tom Lane
Albe Laurenz [EMAIL PROTECTED] writes: You left out the case where --enable_thread_safety is specified. ... If libldap_r.so does require the same libs, please add $EXTRA_LDAP_LIBS to the 'LDAP_LIBS_FE=-lldap_r' line as well. Actually, I did that in the patch-as-committed. It seems to me that

Re: [HACKERS] unixware and --with-ldap

2006-12-14 Thread ohp
[EMAIL PROTECTED] To: Andrew Dunstan [EMAIL PROTECTED] Cc: ohp@pyrenet.fr, pgsql-hackers list pgsql-hackers@postgresql.org Subject: Re: [HACKERS] unixware and --with-ldap Andrew Dunstan [EMAIL PROTECTED] writes: The right way to do this I think is to put an entry adjusting LIBS in src

Re: [HACKERS] unixware and --with-ldap

2006-12-14 Thread Tom Lane
ohp@pyrenet.fr writes: Here's the diff for configure.in that works for me. Applied, thanks. regards, tom lane ---(end of broadcast)--- TIP 1: if posting/reading through Usenet, please send an appropriate

Re: [HACKERS] unixware and --with-ldap

2006-12-13 Thread ohp
On Tue, 12 Dec 2006, Albe Laurenz wrote: Date: Tue, 12 Dec 2006 16:42:50 +0100 From: Albe Laurenz [EMAIL PROTECTED] To: ohp@pyrenet.fr, pgsql-hackers list pgsql-hackers@postgresql.org, [EMAIL PROTECTED] Subject: RE: [HACKERS] unixware and --with-ldap Olivier PRENANT wrote: When I

Re: [HACKERS] unixware and --with-ldap

2006-12-13 Thread ohp
] To: Albe Laurenz [EMAIL PROTECTED] Cc: ohp@pyrenet.fr, pgsql-hackers list pgsql-hackers@postgresql.org Subject: Re: [HACKERS] unixware and --with-ldap Albe Laurenz [EMAIL PROTECTED] writes: Shouldn't that be AC_CHECK_LIB(ldap_r, ldap_simple_bind, [], [AC_MSG_ERROR([library

Re: [HACKERS] unixware and --with-ldap

2006-12-13 Thread Tom Lane
ohp@pyrenet.fr writes: I was in the process of testing this. But I believe tweaking with configure.in will not help me as I have no way to regenerate configure... Install autoconf; it's no big deal. regards, tom lane ---(end of

Re: [HACKERS] unixware and --with-ldap

2006-12-13 Thread Albe Laurenz
Olivier PRENANT wrote: When I swithed to the newest version og pgbuildfarm, I noticed that --with-ldap (now by defaut) didn't work on UnixWare. This is because, on Unixware, ldap needs lber and resolv. Or was libldap not linked against liblber and libresolv? Dunno! Did'nt do it myself,

Re: [HACKERS] unixware and --with-ldap

2006-12-12 Thread ohp
@pyrenet.fr, pgsql-hackers list pgsql-hackers@postgresql.org Subject: Re: [HACKERS] unixware and --with-ldap Andrew Dunstan [EMAIL PROTECTED] 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

Re: [HACKERS] unixware and --with-ldap

2006-12-12 Thread Albe Laurenz
Olivier PRENANT wrote: When I swithed to the newest version og pgbuildfarm, I noticed that --with-ldap (now by defaut) didn't work on UnixWare. This is because, on Unixware, ldap needs lber and resolv. Is libldap a static library on that system? Or do shared libraries on Unixware

Re: [HACKERS] unixware and --with-ldap

2006-12-12 Thread Tom Lane
Albe Laurenz [EMAIL PROTECTED] writes: Shouldn't that be AC_CHECK_LIB(ldap_r, ldap_simple_bind, [], [AC_MSG_ERROR([library 'ldap_r' is required for LDAP])], [$PTHREAD_LIBS $EXTRA_LDAP_LIBS]) Ooops, of course. Like I said, untested ;-)

Re: [HACKERS] unixware and --with-ldap

2006-12-12 Thread Martijn van Oosterhout
On Tue, Dec 12, 2006 at 04:42:50PM +0100, Albe Laurenz wrote: Or do shared libraries on Unixware generally 'not remember' the libraries they were linked against (this would be strange). It could be that whoever compiled libldap there did not include the dependancies at link time. It is legal to

Re: [HACKERS] unixware and --with-ldap

2006-12-11 Thread Andrew Dunstan
ohp@pyrenet.fr wrote: Hi all, When I swithed to the newest version og pgbuildfarm, I noticed that --with-ldap (now by defaut) didn't work on UnixWare. This is because, on Unixware, ldap needs lber and resolv. Not being a configure guru, I made the change bellow locally and that works for me.

Re: [HACKERS] unixware and --with-ldap

2006-12-11 Thread ohp
@postgresql.org Subject: Re: [HACKERS] unixware and --with-ldap ohp@pyrenet.fr wrote: Hi all, When I swithed to the newest version og pgbuildfarm, I noticed that --with-ldap (now by defaut) didn't work on UnixWare. This is because, on Unixware, ldap needs lber and resolv. Not being

Re: [HACKERS] unixware and --with-ldap

2006-12-11 Thread Tom Lane
Andrew Dunstan [EMAIL PROTECTED] 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