Newsgroups: php.internals
Path: news.php.net
Xref: news.php.net php.internals:103142
To: PHP internals <internals@lists.php.net>
References: <8bead512-6032-7cc0-156e-bf2425663cd3@gmx.de>
Message-ID: <72d66cc9-7835-faf4-186b-86dcda23b883@gmx.de>
Date: Sat, 8 Sep 2018 14:04:57 +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: <8bead512-6032-7cc0-156e-bf2425663cd3@gmx.de>
Content-Type: text/plain; charset=utf-8
Content-Language: de-DE
Content-Transfer-Encoding: 8bit
Subject: Re: Lift ICU requirements
From: cmbecker69@gmx.de ("Christoph M. Becker")

On 03.09.2018 at 15:12, Christoph M. Becker wrote:

> ext/intl presently requires ICU ≥ 4.0 (ICU 4.0 has been released on
> 2008-07-02[1]).  Since ICU is still under vivid development our code is
> full of version checks (grep for ICU_VERSION), very old ICU versions are
> unlikely to be still around widespreadly, and we're generally not doing
> us a favor in supporting such old ICU versions (many PHPTs have multiple
> variants for different ICU versions).  Therefore I have submitted PR
> #3487[2] which would raise the requirements to ICU ≥ 4.6 (ICU 4.6 has
> been released on 2010-12-02[3]).
> 
> Anatol suggested to consider to lift the requirements even to ICU ≥ 50
> (ICU 50.1 has been released on 2012-11-05[4]), or maybe even to ICU ≥ 52
> (ICU 52.1 has been released on 2013-10-09[5]).
> 
> What do you think?  Does this require an RFC?
> 
> [1] <http://icu-project.org/download/4.0.html#ICU4C>
> [2] <https://github.com/php/php-src/pull/3487>
> [3] <http://site.icu-project.org/download/46#ICU4C-Download>
> [4] <http://site.icu-project.org/download/50#TOC-ICU4C-Download>
> [5] <http://site.icu-project.org/download/52#TOC-ICU4C-Download>

It appears there is consensus on requiring ICU ≥ 50.1 for master (PHP
7.4).  Unless there are objections, I'll merge PR #3487 (which I've
already amended for ICU 50.1) on the next weekend (i.e. in roughly a week).

-- 
Christoph M. Becker