Re: [PHP-DEV] Lift ICU requirements

2018-09-04 Thread Andrey Andreev
Hi, On Tue, Sep 4, 2018 at 7:22 PM, Sara Golemon wrote: > On Tue, Sep 4, 2018 at 6:44 AM, Andrey Andreev wrote: >> On Mon, Sep 3, 2018 at 6:13 PM, Christoph M. Becker >> wrote: >>> On 03.09.2018 at 17:02, Andrey Andreev wrote: >>> I raised this issue back in January[1], but then got distr

Re: [PHP-DEV] Lift ICU requirements

2018-09-04 Thread Sara Golemon
On Tue, Sep 4, 2018 at 6:44 AM, Andrey Andreev wrote: > On Mon, Sep 3, 2018 at 6:13 PM, Christoph M. Becker 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. >> >>

Re: [PHP-DEV] Lift ICU requirements

2018-09-04 Thread 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 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. >> >> Thank

Re: [PHP-DEV] Lift ICU requirements

2018-09-04 Thread Andrey Andreev
Hi, On Mon, Sep 3, 2018 at 6:13 PM, Christoph M. Becker 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 cha

RE: [PHP-DEV] Lift ICU requirements

2018-09-03 Thread Anatol Belski
> -Original Message- > From: Remi Collet > 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 pr

Re: [PHP-DEV] Lift ICU requirements

2018-09-03 Thread Christoph M. Becker
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

Re: [PHP-DEV] Lift ICU requirements

2018-09-03 Thread Andrey Andreev
Hi, I raised this issue back in January[1], but then got distracted and completely forgot about it ... sorry about that. Would the change target 7.3 or 7.4? [1] https://externals.io/message/101626 Cheers, Andrey. -- PHP Internals - PHP Runtime Development Mailing List To unsubscribe, visit: h

Re: [PHP-DEV] Lift ICU requirements

2018-09-03 Thread Remi Collet
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 s