Re: [HACKERS] Fix linking of OpenLDAP libraries

Tom Lane wrote on September 04, 2006:
> "Albe Laurenz" <all(at)adv(dot)magwien(dot)gv(dot)at> writes:
>
>> # The backend doesn't need everything that's in LIBS, however
>> ! LIBS := $(filter-out -lz -lreadline -ledit -ltermcap -lncurses
-lcurses -lldap_r $(PTHREAD_LIBS), $(LIBS))
>
> This seems pretty risky. What if PTHREAD_LIBS contains -L switches?
> They'd get removed even if needed for other libraries.
>
> It would probably be safer not to put LDAP into LIBS at all, but
invent
> two new macros for configure to set, say LDAP_LIBS and LDAP_LIBS_R,
> and add these to the link lines in the backend and libpq respectively.

That seems like a good idea.
I'll try to work it out and resubmit the fix.

Yours,
Laurenz Albe