On Thu, 5 Jun 2025 18:30:40 GMT, Naoto Sato <na...@openjdk.org> wrote:

>> Changes to generate CLDR resource bundles in UTF-8 encoding. The resource 
>> files in `java.base` are supposed to be US English only, but they also need 
>> to use UTF-8 as some of the names are non-ASCII (e.g., Türkiye)
>
> Naoto Sato has updated the pull request incrementally with one additional 
> commit since the last revision:
> 
>   java.base/Gensrc.gmk comment

Verified that
1. `TOOL_CLDRCONVERTER` points to `CLDRConverter`
2. `CLDRConverter` indeed accepts `-utf8`
3. `make/modules/java.base/Gensrc.gmk` and 
`make/modules/jdk.localedata/Gensrc.gmk` are the only two places where 
`CLDRConverter` is invoked

make/modules/jdk.localedata/Gensrc.gmk line 49:

> 47:       -o $(GENSRC_DIR) \
> 48:       -tzdatadir $(TZ_DATA_DIR) \
> 49:       -utf8)

@naotoj, I assume you did not add the comment to this change intentionally.

-------------

Marked as reviewed by vyazici (Committer).

PR Review: https://git.openjdk.org/jdk/pull/25648#pullrequestreview-2903974963
PR Review Comment: https://git.openjdk.org/jdk/pull/25648#discussion_r2131625005

Reply via email to