Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:74990
Mailing-List: contact internals-help@lists.php.net; run by ezmlm
Received-SPF: pass (pb1.pair.com: domain gmail.com designates 209.85.216.180 as permitted sender)
MIME-Version: 1.0
In-Reply-To: <54FA010D-4FD7-48C9-8268-C1EF8D0B742F@ez.no>
References: <CAH-PCH6xEXspisLOHtiTbViA4vDTk7q6LM3t_Bh9Dzr1cj_Nag@mail.gmail.com>
	<53A1C722.9060501@fedoraproject.org>
	<53A21137.6010705@sugarcrm.com>
	<CAMUwpuT_S2EE6cdJM95RzY4vmORGx3vJ9+gau8G3EZCMsNSydA@mail.gmail.com>
	<53A2A9BD.1070603@sugarcrm.com>
	<54FA010D-4FD7-48C9-8268-C1EF8D0B742F@ez.no>
Date: Thu, 19 Jun 2014 14:58:18 +0200
Message-ID: <CAH-PCH4u10tx=-iXhp_PHYLq9x24PLuAkFsweYv1qnnzOAX8+w@mail.gmail.com>
To: =?UTF-8?B?QW5kcsOpIFLDuG1ja2U=?= <andre.romcke@ez.no>
Cc: Stas Malyshev <smalyshev@sugarcrm.com>, Julien Pauli <jpauli@php.net>, 
	Remi Collet <remi@fedoraproject.org>, PHP Internals <internals@lists.php.net>
Content-Type: multipart/alternative; boundary=047d7bdc8ac452816404fc2fee13
Subject: Re: [PHP-DEV] Problems with the fix for the BC break introduced in
 5.4.29 and 5.5.13
From: tyra3l@gmail.com (Ferenc Kovacs)

--047d7bdc8ac452816404fc2fee13
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: quoted-printable

On Thu, Jun 19, 2014 at 2:04 PM, Andr=C3=A9 R=C3=B8mcke <andre.romcke@ez.no=
> wrote:

> Travis is now running PHP 5.4.29, tests are failing all over the place
> right now*.
> When these kind of things happen, just take down the release while you
> work out the details please.
>
>
> (* most people in the wild don=E2=80=99t always use latests PHPUnit as it=
 has a
> tendency to break stuff badly, just look to the now dead Zeta Components
> for a example of bad PHPunit breaks)
>
>
>
we can't really "take down" a release after it was announced(it is not a
technical limitation, but a logical one, the cat is out of the bag already
at that point), our only option is to quickly release another one, but that
should be used as a last-resort action, as we have many precedence that
releases like that (made under time pressure) are more likely to don't
properly fix the problem at hand, or that it introduces another one.
as you can see from the discussion above, we are still not sure about
whether or not we want to "fix" the behavior change in 5.4.29/5.5.13 so I
think it was a good thing that we didn't took down the release and rushed
another release, which reverts the behavior when there is a chance that we
decide to keep the original fix anyways.

(* most people testing their php lib/app on travis probably use the phpunit
pre-installed on the travis box, the same as they do with the php install)

ps: please don't top post on this list.

--=20
Ferenc Kov=C3=A1cs
@Tyr43l - http://tyrael.hu

--047d7bdc8ac452816404fc2fee13--