Hi Erik,

Yep, that is not needed so I will remove it. Thank you for catching it.

Naoto

On 10/24/14, 12:28 AM, Erik Joelsson wrote:
Hello Naoto,

Just one thing, in Gensrc-java.base.gmk, shouldn't we still just run the
targets in GENSRC_JAVA_BASE and let Gensrc-jdk.localedata.classic.gmk
deal with GENSRC_JDK_LOCALEDATA_CLASSIC?

/Erik

On 2014-10-23 22:05, Naoto Sato wrote:
I revised the fix according to the suggestions I got. The main change
is to rename the name of the locale data modules. Now we have two as
follows:

- jdk.localedata.classic: Legacy locale data
- jdk.localedata.cldr: CLDR locale data

Revised webrev is located at:

http://cr.openjdk.java.net/~naoto/8061382/webrev.01/

Naoto

On 10/17/14, 3:31 PM, Naoto Sato wrote:
Hello,

Please review the proposed changes to the following bug:

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

The webrev is available at:

http://cr.openjdk.java.net/~naoto/8061382/webrev.00/

This is mainly build changes to separate CLDR locale data into a new
module. All those CLDR source changes (for .xml files) are merely the
location changes in the repository, no changes are made to their file
contents.

Naoto

Reply via email to