[9] RFR: 8058509: CLDRLocaleDataMetaInfo should be in jdk.localedata

[Erik Joelsson]

On 2014-09-16 23:03, Naoto Sato wrote:
> I revised the fix based on suggestions from Erik/Magnus. I just ended 
> up creating Gensrc-jdk.localedata.gmk, instead of renaming 
> GensrcCLDR.gmk because GensrcLocaleData.gmk (formerly 
> GensrcLocaleDataMetaInfo.gmk) is also needed to build jdk.localedata:
>
> http://cr.openjdk.java.net/~naoto/8058509/webrev.1/
>
Right, you were able to separate the work for java.base and 
jdk.localedata in GensrcLocaleData.gmk. The drawback is that the shell 
find and calculation of the locale lists will be executed twice, but I 
suppose we can live with that. It's good to get a clean separation of 
gensrc between different modules when possible.

Looks good to me now.

/Erik
> Please review.
>
> Naoto
>
> On 9/15/14, 4:30 PM, Naoto Sato wrote:
>> Hello,
>>
>> Please review the fix for the following issue:
>>
>> https://bugs.openjdk.java.net/browse/JDK-8058509
>>
>> The webrev is located at:
>>
>> http://cr.openjdk.java.net/~naoto/8058509/webrev.0/
>>
>> The fix is intended to move the LocaleDataMetaInfo of CLDR from
>> java.base module to jdk.localedata module, along with CLDR's "en" locale
>> data and the package name change to avoid the "split package" in two
>> modules.
>>
>> Naoto