On Mon, 5 Feb 2024 10:38:26 GMT, Magnus Ihse Bursie <i...@openjdk.org> wrote:
>> We're currently setting LC_ALL=C. Not all tools will default to utf-8 as >> their encoding of choice when they see this locale, but use an arbitrarily >> encoding, which might not properly handle all UTF-8 characters. Since in >> practice, all our encoding is utf8, we should tell our tools this as well. >> >> This will at least have effect on how Java treats path names including >> unicode characters. > > Magnus Ihse Bursie has updated the pull request incrementally with two > additional commits since the last revision: > > - Update copyright year > - check for utf-8 first Marked as reviewed by erikj (Reviewer). make/autoconf/basic.m4 line 136: > 134: fi > 135: else > 136: AC_MSG_WARN([locale command not not found, using C locale]) Inconsistent indentation here. ------------- PR Review: https://git.openjdk.org/jdk/pull/16971#pullrequestreview-1862818344 PR Review Comment: https://git.openjdk.org/jdk/pull/16971#discussion_r1478262965