Newsgroups: php.internals,php.qa
Path: news.php.net
Xref: news.php.net php.internals:36730 php.qa:64137
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: unknown (pb1.pair.com: domain php.net does not designate 83.243.58.163 as permitted sender)
To: Marcus Boerger <helly@php.net>
Cc: Derick Rethans <derick@php.net>, Marco Kaiser <marco.kaiser@gmail.com>,  'PHP Developers Mailing List' <internals@lists.php.net>, php-qa@lists.php.net, Ilia Alshanetsky <ilia@prohost.org>
In-Reply-To: <826883759.20080331171437@marcus-boerger.de>
References: <a2b4d2820803270125w1c167b3fmbd721458457f3643@mail.gmail.com>
	 <1124764115.20080327105033@marcus-boerger.de>
	 <47eb9d50.02e2660a.759d.2e19@mx.google.com>
	 <alpine.DEB.0.98.0803301729300.2627@kossu.ez.no>
	 <826883759.20080331171437@marcus-boerger.de>
Content-Type: text/plain
Date: Mon, 31 Mar 2008 23:18:34 +0200
Message-ID: <1206998314.2941.6.camel@goldfinger>
Mime-Version: 1.0
Content-Transfer-Encoding: 7bit
Subject: Re: [PHP-QA] BC break with php 5.2.6
From: johannes@php.net (Johannes =?ISO-8859-1?Q?Schl=FCter?=)

Marcus,

On Mon, 2008-03-31 at 17:14 +0200, Marcus Boerger wrote:
> Hello Derick,
> 
>   it is a BUG FIX. Not fixing it means more people will turn it into a
> feature and we will never be able to remove it. Becuase next time you
> will say we cannot remove it in a minor version as by then you accidentally
> rely on it yourself.
> 
> We made a mistake! Lets stand up and admit we did. Probably I was involved
> in making that mistake, so sorry. Done.

I have to agree with Derick here we shouldn't add a fatal error within a
bug fix release at such a place. People (hosters...) tend to update
versions without checking all incompatibilities - since you don't expect
them and they want to get security issues fixed.

Yes, the previous behavior was wrong! Yes we shouldn't encourage people
do do that.

I'd propose making it an Warning or something in 5.2 and fixing it in
5.3.

johannes