On Fri, 23 Feb 2024 21:24:10 GMT, Naoto Sato <na...@openjdk.org> wrote:
> This PR intends to remove the legacy `COMPAT` locale data from the JDK. The > `COMPAT` locale data was introduced for applications' migratory purposes > transitioning to `CLDR`. It is becoming a technical debt and now is the time > to remove it (we've been emitting a warning at JVM startup since JDK21, if > the app is using `COMPAT`). A corresponding CSR has also been drafted. make/modules/java.base/gensrc/GensrcLocaleData.gmk line 58: > 56: ifeq ($(MODULE), jdk.localedata) > 57: $(shell $(RM) > $(SUPPORT_OUTPUTDIR)/gensrc/jdk.localedata/sun/util/resources/cldr/provider/CLDRLocaleDataMetaInfo_jdk_localedata.java) > 58: endif The remainder of this file doesn't seem to be doing anything useful. Do we really need to keep it around? If I understand this correctly, the files referenced and deleted here are generated by the cldrconverter. Is that build tool relying on make removing the file first? Reading the java source, my impression is that it will generate the file unconditionally regardless of the files current existence as long as the tool is run. ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/17991#discussion_r1502691516