BUG #3961: ecpg lacks SQLSTATE macro definition

Started by Christoph Dalitzabout 18 years ago4 messagesbugs
Jump to latest
#1Christoph Dalitz
christoph.dalitz@hs-niederrhein.de

The following bug has been logged online:

Bug reference: 3961
Logged by: Christoph Dalitz
Email address: christoph.dalitz@hs-niederrhein.de
PostgreSQL version: 8.2.6
Operating system: Linux
Description: ecpg lacks SQLSTATE macro definition
Details:

While ecpg defines the (deprecated) SQLCODE, it does not define the macro
SQLSTATE, even though the documentation say, that this latter macro should
be used and which values it can take.

Please add a macro definition

#define SQLSTATE sqlca.sqlstate

in ecpglib.h like for SQLSTATE to be compatible with the SQL standard.

Thanks,

Christoph

#2Michael Meskes
meskes@postgresql.org
In reply to: Christoph Dalitz (#1)
Re: BUG #3961: ecpg lacks SQLSTATE macro definition

Hallo Christoph,

On Thu, Feb 14, 2008 at 11:09:11AM +0000, Christoph Dalitz wrote:

Description: ecpg lacks SQLSTATE macro definition

Thanks for reporting this oversight. I added the definition to CVS, also
for 8.2.

Michael
--
Michael Meskes
Email: Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org)
ICQ: 179140304, AIM/Yahoo: michaelmeskes, Jabber: meskes@jabber.org
Go VfL Borussia! Go SF 49ers! Use Debian GNU/Linux! Use PostgreSQL!

#3Tom Lane
tgl@sss.pgh.pa.us
In reply to: Michael Meskes (#2)
Re: BUG #3961: ecpg lacks SQLSTATE macro definition

Michael Meskes <meskes@postgresql.org> writes:

Hallo Christoph,
On Thu, Feb 14, 2008 at 11:09:11AM +0000, Christoph Dalitz wrote:

Description: ecpg lacks SQLSTATE macro definition

Thanks for reporting this oversight. I added the definition to CVS, also
for 8.2.

Michael, we branched 8.3 yesterday, so you need a third commit if you
don't want people to see a regression from 8.2.7 to 8.3.x.

regards, tom lane

#4Michael Meskes
meskes@postgresql.org
In reply to: Tom Lane (#3)
Re: BUG #3961: ecpg lacks SQLSTATE macro definition

On Thu, Feb 14, 2008 at 11:13:53AM -0500, Tom Lane wrote:

Michael, we branched 8.3 yesterday, so you need a third commit if you
don't want people to see a regression from 8.2.7 to 8.3.x.

Thanks for the note, just committed to 8.3 branch as well.

Michael
--
Michael Meskes
Email: Michael at Fam-Meskes dot De, Michael at Meskes dot (De|Com|Net|Org)
ICQ: 179140304, AIM/Yahoo: michaelmeskes, Jabber: meskes@jabber.org
Go VfL Borussia! Go SF 49ers! Use Debian GNU/Linux! Use PostgreSQL!