RFR: JDK-8204973: Add build support for filtering translations
naoto.sato at oracle.com
Thu Jun 14 20:05:15 UTC 2018
Looks good to me.
On 6/14/18 11:52 AM, Erik Joelsson wrote:
> Here is a new version of the patch:
> Changes from last time:
> * Made the regexp for finding locales more correct. It still does not
> try to match 3 letter language strings because doing so triggers a large
> amount of false positives in our souce tree.
> * Added another accepted locale (en_US_POSIX) that is now matched by the
> improved regexp.
> * Added more locales to the exclude list as they were now discovered by
> the improved regexp.
> On 2018-06-13 12:47, Erik Joelsson wrote:
>> Oracle will reduce the number of languages that it maintains
>> translations of JDK resources for. The current translations will
>> remain in the source for now, but we need a way to filter out a set of
>> translations at build time so that we only include the ones we
>> support. This patch adds such a configuration option. It also changes
>> how Oracle builds by using the option to exclude all translations
>> except English, Japanese, Simplified Chinese and Traditional Chinese.
>> Anyone else building OpenJDK will by default include all translations
>> present in the source, just as before.
>> I added a test that verifies this for builds with the "IMPLEMENTOR"
>> field in the release file set to "Oracle Corporation". The test will
>> not be run for other OpenJDK builds.
>> I had to modify an existing test for java.logging which used various
>> translations to verify localized log messages to only use translations
>> that Oracle chooses to include.
>> Since this is the second test that specifically verifies build
>> behavior, I moved the previous such test together with this new test
>> into a common top level test directory "build", under the jdk test
>> root. I put these tests in the jdk tier3 test group.
>> I have run all tier1, 2 and 3 tests in Mach 5 as well as specifically
>> looked for tests that use the java.util.Locale class and ran them
>> Webrev: http://cr.openjdk.java.net/~erikj/8204973/webrev.01/index.html
>> Bug: https://bugs.openjdk.java.net/browse/JDK-8204973
More information about the core-libs-dev