Re: Lift ICU requirements

Hi, On Tue, Sep 4, 2018 at 7:22 PM, Sara Golemon <pollita@php.net> wrote: > On Tue, Sep 4, 2018 at 6:44 AM, Andrey Andreev <narf@devilix.net> 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. :) >> > Yeah, I recall that discussion and I don't recall any strong > objections. +1 to lifting the bar. If you're using latest PHP, then > you can be expected to have reasonably-recent ICU. > >>>> Would the change target 7.3 or 7.4? >>> >>> IMO, we should target master (i.e. 7.4) only. >>> >> > Concur. Feature freeze means feature freeze. Nothing stops anyone > from using a more recent ICU with 7.3. There's no urgency > justification for overriding FF here. > >> 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. >> > No. See above. There is no urgency justification for it. > Right ... I wasn't saying it's urgent, just wasn't sure whether FF rule applies. If it does, I'm not trying to bend it. Cheers, Andrey.