Re: patch to fix configure(.in) on openbsd wrt/ krb5/com_err and readline linking - Mailing list pgsql-bugs

From Greg Stark
Subject Re: patch to fix configure(.in) on openbsd wrt/ krb5/com_err and readline linking
Date
Msg-id 4136ffa0906100619l124a7d4ve8fbbd154d884d3d@mail.gmail.com
Whole thread Raw
In response to patch to fix configure(.in) on openbsd wrt/ krb5/com_err and readline linking  (Simon Bertrang <janus@errornet.de>)
Responses Re: Re: patch to fix configure(.in) on openbsd wrt/ krb5/com_err and readline linking
Re: patch to fix configure(.in) on openbsd wrt/ krb5/com_err and readline linking
List pgsql-bugs
On Wed, Jun 10, 2009 at 10:25 AM, Simon Bertrang<janus@errornet.de> wrote:
>
> the following patch does two things on OpenBSD:

Thank you.


> =A01) Add missing libs to the krb5/com_err check that are required.
> =A0 =A0We have this in our ports tree since 7.4.3 but i can't find any
> =A0 =A0report about it, so here it finally is.

This seems really weird. Firstly, doesn't OpenBSD use ELF? Shouldn't
the library pull in the indirectly needed libraries automatically? But
more to the point, why on *earth* would com_err depend on -lssl and
-lcrypto? com_err is just a standard error handling library.  Why
would it fail to link without ssl and crypto libraries?!?!

If it is necessary, putting in an "if portname =3D openbsd" is defeating
the whole purpose of using autoconf here. Surely something like

 =A0 =A0AC_SEARCH_LIBS(com_err, [krb5 'krb5 -lcrypto -ldes -lasn1 -lroken'
'com_err' 'com_err -lssl -lcrypto'], [],

would be better since it would detect this situation regardless of
what OS it's on.


--=20
Gregory Stark
http://mit.edu/~gsstark/resume.pdf

pgsql-bugs by date:

Previous
From: hubert depesz lubaczewski
Date:
Subject: Re: Postgres
Next
From: Tom Lane
Date:
Subject: Re: BUG #4838: Database corruption after btree_gin index creation