Dmitry,
The registry patch can stay, but please revert the soap and
zend_compile changes until Friday.
On 2-May-07, at 12:17 PM, Dmitry Stogov wrote:
Hi Ilia,
The registry.c definitely must not be reverted. It fixes serious
regression
since 5.1.6.
I can revert ext/soap and zend_compile.c fixes that I made today to
not
delay release process, but I don't think they may break more than
zend_fcall_info_init() fix by Johannes. So it probably should be
reverted
too.
Could you send something like "CVS IS CLOSED BECAUSE OF PHP-?.?.?
RELEASE
PROCESS" into internals@ next times.
It would save our time and nerves.
Thanks. Dmitry.
-----Original Message-----
From: Ilia Alshanetsky [mailto:[EMAIL PROTECTED]
Sent: Wednesday, May 02, 2007 5:25 PM
To: Stanislav Malyshev
Cc: Antony Dovgal; Dmitry Stogov; php-cvs@lists.php.net
Subject: Re: [PHP-CVS] cvs: php-src(PHP_5_2) / NEWS /win32 registry.c
Let's leave this one in then, but please revert the other changes.
5.2.2 will be packed today, so by Friday the changes can go
back into
the CVS.
On 2-May-07, at 9:11 AM, Stanislav Malyshev wrote:
Regression from 5.2.1 or earlier?
As far as I understand, it's been broken in 5.2.1 and this patch
fixes it.
From what I understand, this patch fixes broken registry .ini
control in PHP on windows. It seems to eb quite isolated and if I
understand the nature of the bug correctly without the fix the
registry per-dir entries don't work. So it might be a
reason to let
this in as an exception.
--
Stanislav Malyshev, Zend Products Engineer
[EMAIL PROTECTED] http://www.zend.com/
Ilia Alshanetsky
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
Ilia Alshanetsky
--
PHP CVS Mailing List (http://www.php.net/)
To unsubscribe, visit: http://www.php.net/unsub.php
