Re: PHP 7.2 minimum ICU version

Hi! On 22.08.2019 at 01:17, Trevor Rowbotham wrote: > I realize that this may be a little late given that PHP 7.2 initially released > roughly 2 years ago, but I came across an issue yesterday regarding the minimum > required version of the ICU library. The user's hosting provider was using > PHP 7.2.1, but it was linked with an ICU version of 4.2.1 (over 10 years old!). > > According to the > https://wiki.php.net/rfc/deprecate-and-remove-intl_idna_variant_2003 > RFC, the INTL_IDNA_VARIANT_2003 constant was deprecated in PHP 7.2. However, this > RFC did not increase the minimum required version of ICU, which is currently > 4.0 according to > https://www.php.net/manual/en/intl.requirements.php. This > leaves an opening where PHP 7.2 can be built with an ICU version less than 4.6, > but the user cannot avoid the deprecation notice because the replacement, > INTL_IDNA_VARIANT_UTS46, wasn't added until ICU 4.6. > > Is there any way to increase the minimum required ICU version in PHP 7.2 and 7.3 to > at least 4.6, which would ensure that the upgrade path is actually available to users > trying to avoid the deprecation notice? We usually do not raise any dependency requirements for stable release branches, to avoid potential BC breaks. And after all, most of ext/intl is supposed to work flawlessly with such ancient ICU versions, and to be able to get rid of this deprecation notice, building with a newer ICU version is already possible (and, in my opinion, recommendable anyway). Thanks, Christoph