Re: [HACKERS] Re: unixODBC again :-( - Mailing list pgsql-odbc

From Peter Eisentraut
Subject Re: [HACKERS] Re: unixODBC again :-(
Date
Msg-id Pine.LNX.4.30.0101241908170.1469-100000@peter.localdomain
Whole thread Raw
In response to Re: unixODBC again :-(  (Bruce Momjian <pgman@candle.pha.pa.us>)
List pgsql-odbc
Bruce Momjian writes:

> > 1. Add options to use unixODBC in the configure.in file, the mainly consists of finding the root of the unixODBC
installprefix, and adding 
> > -I /unixODBC/path/include and -L /unixODBC/path/lib to the driver build

--with-includes, --with-libraries

> > 2. Change the way the driver gets config info, to be the same as when built under windows. link with -lodbcinst and
itprovides 
> > SQLGetPrivateProfileString. the code that calls this works as long as the correct define is set.

I don't understand this.  The driver gets the config info just fine; why
add another way?

> > 3. Stop calling ODBC functions in the driver, this is simple but messy, the problem being the call (say) in
SQLAllocStmtthat calls 
> > SQLAllocHandle in the driver, ends up calling the SQLAllocHandle in the driver manager.

This is fixed using magic linker options on ELF platforms.  I don't recall
how the patch tried to address this, but a better solution is probably
necessary.

> > There are a couple of other changes I have made, that you may want
> to add, I added the code to allow encrypted passwords (taken from the
> pg > lib), as crypt is avaiable on unix.

Why not.

> Add the option to detect a
> server name of localhost, and open the unix domain socket,

I don't think so.  localhost is a valid host name.

> in fact try
> two > places, to handle the debian build where the location is
> different.

We have a general approach to non-standard socket names now.

--
Peter Eisentraut      peter_e@gmx.net       http://yi.org/peter-e/


pgsql-odbc by date:

Previous
From: Tom Lane
Date:
Subject: Re: [PATCHES] ODBC Patch for OJs/Large Querys & Rows
Next
From: Nick Gorham
Date:
Subject: Re: [HACKERS] Re: unixODBC again :-(