Thanks, Alan.

I thought of the same for en_US w/o jdk.localedata case, and reached the same conclusion.

I found one typo in my fix, where "-addmods" in plugins.properties should be "--addmods". I will correct it and push the changeset.

Naoto

On 5/4/16 3:03 AM, Alan Bateman wrote:
On 04/05/2016 00:33, Naoto Sato wrote:
Hi,

Please review the fix for the following issue:

https://bugs.openjdk.java.net/browse/JDK-8155649

The proposed fix is located at:

http://cr.openjdk.java.net/~naoto/8155649/webrev.00/
This looks okay. There is a corner case where it should be possible to
--include-locales to specify en_US and it should always succeed but I
don't think that is worth spending time on.

-Alan

Reply via email to