Re: [PHP-DEV] Unification of phpinfo() output for PDO drivers

2007-11-09 Thread Stanislav Malyshev
Attached is an updated patch that covers all drivers in HEAD. Why this patch says "PECL module version" when the modules are actually not PECL modules? Did you mean "PDO module version"? -- Stanislav Malyshev, Zend Software Architect [EMAIL PROTECTED] http://www.zend.com/ (408)253-8829 MSN

Re: [PHP-DEV] Unification of phpinfo() output for PDO drivers

2007-10-14 Thread Marcus Boerger
Hello Martin, using the $Revision: $ keyword is enough. Also it looks like you're having a white space issue. marcus Friday, October 12, 2007, 5:54:53 PM, you wrote: > Index: pdo_mysql/pdo_mysql.c > === > RCS file: /repository/

Re: [PHP-DEV] Unification of phpinfo() output for PDO drivers

2007-10-13 Thread Martin Jansen
On Fri Oct 12, 2007 at 01:5229PM -0700, Stanislav Malyshev wrote: > Martin Jansen wrote: > >The attached patch against HEAD attempts to unify the phpinfo() output > >of the PDO drivers for Oracle, MySQL, and PostgreSQL by always including > >the PECL module version string and the $Id$ CVS keyword.

Re: [PHP-DEV] Unification of phpinfo() output for PDO drivers

2007-10-12 Thread Stanislav Malyshev
I think this would be a good idea to add these. Martin Jansen wrote: The attached patch against HEAD attempts to unify the phpinfo() output of the PDO drivers for Oracle, MySQL, and PostgreSQL by always including the PECL module version string and the $Id$ CVS keyword. (Akin to how it is done f

[PHP-DEV] Unification of phpinfo() output for PDO drivers

2007-10-12 Thread Martin Jansen
The attached patch against HEAD attempts to unify the phpinfo() output of the PDO drivers for Oracle, MySQL, and PostgreSQL by always including the PECL module version string and the $Id$ CVS keyword. (Akin to how it is done for the SQLite driver.) Assuming that you are interested in the changes,