Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:16032
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Content-Class: urn:content-classes:message
MIME-Version: 1.0
Content-Type: text/plain;
	charset="us-ascii"
Content-Transfer-Encoding: quoted-printable
Date: Tue, 19 Apr 2005 14:53:55 +0100
Message-ID: <08237065FA027340B731E57099097854039A9C58@ukslms22.ca.com>
Thread-Topic: [PATCH] Ingres error reporting functions
Thread-Index: AcVE5zhA6dYyBtorQKaTaBuvHG9OlQ==
To: <internals@lists.php.net>
Subject: [PATCH] Ingres error reporting functions
From: Grant.Croker@ca.com ("Croker, Grant")


Hi,

I would like to submit the following patch to PHP:
http://www.ingres.co.uk/php/head/ingres_error_reporting.diff

It includes 3 new functions for reporting errors; ingres_error(),
ingres_error_text() and ingres_error_sqlstate(). Also included is the
ability to suppress E_WARNING messages that happen when a database error
is detected. This requires the following to be added to php.ini under
the section titled "[Ingres II]":

; Report database warnings or not
ingres.report_db_warnings =3D 0

When trying to test this functionality on Linux I ran across a
problem/difference in the way Windows and UNIX/Linux based autoconf
generates the Makefile. It would appear that with UNIX it expects its
configuration files be located in a directory name equivalent to the
first parameter to PHP_NEW_EXTENSION in config.m4.

For most extensions I guess this not a problem since the directory and
extension name match up. For the Ingres extension they don't. After the
fix for bug 32333/32359 the extension name changed from ingres_ii to
ingres to allow the extension to build on Windows. The fix was needed as
the Windows build process uses the extname in 'ARG_WITH(extname,...)' to
work out what the extension is called. This had the unforeseen affect of
breaking the UNIX/Linux configure process.=20

I guess extension names and their relative 'ext' dir should match? Can
the directory ext/ingres_ii be moved to ext/ingres?

Regards,

grant

--
Grant Croker
Software Engineer, Ingres Development
mailto:grant.croker at ca.com