[11u] 8215913: [Test_bug]java/util/Locale/LocaleProvidersRun.java failed on de_DE and ja_JP locale.

Langer, Christoph christoph.langer at sap.com
Mon Oct 7 14:46:50 UTC 2019


Hi Severin,

> > Your webrev looks fine but doesn't contain the change below to
> > LocaleProviders.sh. This should definitely be done since it is an
> > important part of the original change although being part of
> > LocaleProvidersRun.java instead of LocaleProviders.sh. Can you update
> > the webrev? Then I can put it into our queue for regression testing.
> 
> The change below is the *antidelta* of what's been done in the original
> changeset. JDK 11u's version of LocaleProviders.sh already has the
> equivalent of LocaleProvidersRun.java change:
> 
> if [ "${DEFFMTLANG}" = "zh" ] && [ "${DEFFMTCTRY}" = "CN" ]; then
> ...
> 

You're right, sorry - I was to blind to see ��

So from my end, your patch is ok as proposed. We should push it for parity.

Thanks
Christoph



More information about the jdk-updates-dev mailing list