Re: cvs: php-src(PHP_5_2) / NEWS configure.in /main php_version.h

Moriyoshi, Well, the only thing we could do at this point is to re-tag the release with the change delaying the release by a week. You know where I stand on the matter of the code default value, but if there is a consensus on internal that this change needs to happen, then this can definitely be done. However, looking at the past discussion on internals between Andrei and yourself I don't see anyone objecting to the current implementation, so I see no issue with proceeding as is. Let's reach a conclusion by end of day (EST time) so release can either be made or delayed. On 25-Feb-09, at 11:29 PM, Moriyoshi Koizumi wrote:

In addition, I didn't insist that the whole thing should be reverted. I just argue about the default value. Moriyoshi On Thu, Feb 26, 2009 at 1:27 PM, Moriyoshi Koizumi <mozo@mozo.jp> wrote:

The issue is not whether it is valid or not. Andrei's patch broke the backwards compatibility that affects real-world applications and thus cannot be regarded as an improvement. Moriyoshi On Thu, Feb 26, 2009 at 11:51 AM, Ilia Alshanetsky <ilia@prohost.org> wrote:

Moriyoshi, My opinion is that the current implementation is valid, which is why Andrei's patch was accepted in the first place. On 25-Feb-09, at 9:34 PM, Moriyoshi Koizumi wrote:

Ilia, We didn't resolve the array_unique() issue yet. Please take this into account. Moriyoshi 2009/2/25 Ilia Alshanetsky <iliaa@php.net>:

iliaa           Wed Feb 25 14:56:51 2009 UTC

 Modified files:              (Branch: PHP_5_2)
  /php-src    configure.in NEWS
  /php-src/main       php_version.h

Log: 5.2.9 -- PHP CVS Mailing List (http://www.php.net/) To unsubscribe, visit: http://www.php.net/unsub.php

Ilia Alshanetsky

Ilia Alshanetsky