On Fri, May 04, 2007 at 11:26:24AM +0200, Peter Eisentraut wrote:
> Package: unixodbc-dev
> Version: 2.2.11-13
> Severity: wishlist
> Tags: patch

> Please install the program odbc_config in the package unixodbc-dev.  The
> new psqlodbc driver wants this to build.

So, I'm philosophically opposed to the introduction of new per-library
"-config" helpers, because of how often they get things wrong.  Including
odbc_config in this case:

$ ./debian/tmp/usr/bin/odbc_config --static-libs
/usr/lib/libodbc.a
$

That's wrong, a) there's no reason to output the /usr/lib instead of just
outputting -lodbc, and b) it's missing references to -lltdl and -lpthread

$ ./debian/tmp/usr/bin/odbc_config --libtool-libs
/usr/lib/libodbc.la
$

This is an abstraction violation, nothing should be referencing the .la file
directly and I will not ship anything in a Debian package I maintain that
encourages such misuse of libtool internals.

I'll try to find some time to talk to upstream about this, and encourage
them to ship a pkg-config .pc file instead (which I would then encourage
psqlodbc upstream to give preference to over odbc_config).  I'm not sure if
I would agree to ship odbc_config in the meantime if it were fixed to not
give wrong output, I'll have to think about that.

Cheers,
-- 
Steve Langasek                   Give me a lever long enough and a Free OS
Debian Developer                   to set it on, and I can move the world.
[EMAIL PROTECTED]                                   http://www.debian.org/


-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to