> -----Original Message----- > From: Remi Collet <r...@fedoraproject.org> > Sent: Monday, September 3, 2018 3:55 PM > To: internals@lists.php.net > Subject: Re: [PHP-DEV] Lift ICU requirements > > Le 03/09/2018 à 15:12, Christoph M. Becker a écrit : > > Hi! > > > > 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? > > 50.1 seems a good target (version in RHEL/CentOS 7) > The further idea was also to check with a newer C++ version which could bring improvements. In most it would concern the ICU headers compatibility for ICU < 59, but the older the ICU version would be - the less would be the chance to get it working. I'd be going for it after Christoph's patch has landed.
> > Does this require an RFC? > > I don't think. > Same here. Versions older than 50.1 are likely not used widely and the most of tests run also on some more recent versions, thus it's more about the dead code elimination and refactoring. Regards Anatol
