Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:103123
MIME-Version: 1.0
In-Reply-To: <9953cf2a-8ed2-6a3f-4ac2-1f002d61649b@gmx.de>
References: <8bead512-6032-7cc0-156e-bf2425663cd3@gmx.de> <f0ca4191-dd05-d1f8-badc-8e584202d96f@fedoraproject.org>
 <CAPhkiZwMp-T4XQJNpyKLo3q1S39eWuX_5qL1-VRWLNr3L_VFoQ@mail.gmail.com> <9953cf2a-8ed2-6a3f-4ac2-1f002d61649b@gmx.de>
Date: Tue, 4 Sep 2018 14:44:20 +0300
Message-ID: <CAPhkiZx1ifn=hoMtU4y6oT7d=hCreNXMbU_fZ2YTvcL0aWF_7g@mail.gmail.com>
To: "Christoph M. Becker" <cmbecker69@gmx.de>
Cc: Remi Collet <remi@fedoraproject.org>, PHP internals <internals@lists.php.net>
Content-Type: text/plain; charset="UTF-8"
Subject: Re: [PHP-DEV] Lift ICU requirements
From: narf@devilix.net (Andrey Andreev)

Hi,

On Mon, Sep 3, 2018 at 6:13 PM, Christoph M. Becker <cmbecker69@gmx.de> wrote:
> On 03.09.2018 at 17:02, Andrey Andreev wrote:
>
>> I raised this issue back in January[1], but then got distracted and
>> completely forgot about it ... sorry about that.
>
> Thanks for having raised the issue. :)
>
>> Would the change target 7.3 or 7.4?
>
> IMO, we should target master (i.e. 7.4) only.
>
>> [1] https://externals.io/message/101626
>

Well, bummer. Now I feel real guilty about not acting on time,
especially since you yourself were fine with bumping to ICU 4.6 for
PHP 7.2.2 at the time.

I know we're past feature freeze, but can we maybe do 4.6 in 7.3 and
then 50 in 7.4? We already know ICU 4.6 works well and older versions
aren't widespread; it's not like it needs extensive evaluation.

Cheers,
Andrey.