Re: unixware and --with-ldap

From: Bruce Momjian <bruce(at)momjian(dot)us>
To: Albe Laurenz <all(at)adv(dot)magwien(dot)gv(dot)at>
Cc: ohp(at)pyrenet(dot)fr, Tom Lane <tgl(at)sss(dot)pgh(dot)pa(dot)us>, pgsql-hackers list <pgsql-hackers(at)postgresql(dot)org>
Subject: Re: unixware and --with-ldap
Date: 2007-02-03 02:46:04
Message-ID: 200702030246.l132k4608947@momjian.us
Views: Raw Message | Whole Thread | Download mbox | Resend email
Thread:
Lists: pgsql-hackers


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 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 src/templates/unixware
>
> PTHREAD_CFLAGS, of course :^)
>
> Oops, this seems to be an oversight in (my) original code.
>
> I'd say we should change the lines
>
> AC_CHECK_LIB(ldap_r, ldap_simple_bind, [],
> [AC_MSG_ERROR([library 'ldap_r' is required for LDAP])],
> [$PTHREAD_LIBS $EXTRA_LDAP_LIBS])
>
> to
>
> AC_CHECK_LIB(ldap_r, ldap_simple_bind, [],
> [AC_MSG_ERROR([library 'ldap_r' is required for LDAP])],
> [$PTHREAD_CFLAGS $PTHREAD_LIBS $EXTRA_LDAP_LIBS])
>
> I know that this is abuse of AC_CHECK_LIB, but it is only a test
> and LIBS is discarded later.
>
> Yours,
> Laurenz Albe
>
> ---------------------------(end of broadcast)---------------------------
> TIP 6: explain analyze is your friend

--
Bruce Momjian bruce(at)momjian(dot)us
EnterpriseDB http://www.enterprisedb.com

+ If your life is a hard drive, Christ can be your backup. +

In response to

Browse pgsql-hackers by date

  From Date Subject
Next Message Jeremy Drake 2007-02-03 03:01:33 Re: [HACKERS] writing new regexp functions
Previous Message Bruce Momjian 2007-02-03 02:44:27 Re: [HACKERS] unixware and --with-ldap