Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:103124
To: Andrey Andreev <narf@devilix.net>
Cc: Remi Collet <remi@fedoraproject.org>,
 PHP internals <internals@lists.php.net>
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>
 <CAPhkiZx1ifn=hoMtU4y6oT7d=hCreNXMbU_fZ2YTvcL0aWF_7g@mail.gmail.com>
Message-ID: <ab248eef-e8b7-a047-8c97-0c1553df2472@gmx.de>
Date: Tue, 4 Sep 2018 16:32:01 +0200
User-Agent: Mozilla/5.0 (Windows NT 10.0; WOW64; rv:60.0) Gecko/20100101
 Thunderbird/60.0
MIME-Version: 1.0
In-Reply-To: <CAPhkiZx1ifn=hoMtU4y6oT7d=hCreNXMbU_fZ2YTvcL0aWF_7g@mail.gmail.com>
Content-Type: text/plain; charset=utf-8
Content-Language: de-DE
Content-Transfer-Encoding: 8bit
Subject: Re: [PHP-DEV] Lift ICU requirements
From: cmbecker69@gmx.de ("Christoph M. Becker")

On 04.09.2018 at 13:44, Andrey Andreev wrote:

> 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.

I don't see a real issue here.  If anybody using PHP 7.3 (or even 7.2)
still uses a very old ICU, than they get the deprecation warning.  If
they change to INTL_IDNA_VARIANT_UTS46, they quickly notice that it
doesn't work, and likely look up in the PHP manual (where it should
already have been documented that INTL_IDNA_VARIANT_UTS46 required ICU ≥
4.6; I'll catch up on this soon), so they have the choice to upgrade
their ICU, or to stick with the deprecation warning.

Only for PHP 7.4 it is important to require at least ICU 4.6, since
otherwise changing the default of the $variant parameter would not be
possible.

-- 
Christoph M. Becker