<i18n dev> [9] RFR: 8058509: CLDRLocaleDataMetaInfo should be in jdk.localedata

Magnus Ihse Bursie magnus.ihse.bursie at oracle.com
Wed Sep 17 10:48:25 UTC 2014


On 2014-09-17 11:36, Erik Joelsson wrote:
>
> 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.

Looks good to me, too.

/Magnus



More information about the i18n-dev mailing list