RFR: 8321373: Build should use LC_ALL=C.UTF-8 [v2]

[Magnus Ihse Bursie]

On Thu, 1 Feb 2024 13:53:56 GMT, Magnus Ihse Bursie <ihse at 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 with a new target base due to a merge or a rebase. The incremental webrev excludes the unrelated changes brought in by the merge/rebase. The pull request contains three additional commits since the last revision:
> 
>  - Explicitly load StandardCharsets ascii/utf-8 in HelloClasslist
>  - Merge branch 'master' into c.utf-8
>  - 8321373: Build should use LC_ALL=C.UTF-8

Actually, that diff was apparently platform specific. I'm not quite sure if that is caused by a different set of classes already being included in the different platforms. The above list was from macOS. On Linux, the diff look like this:


> java/nio/StringCharBuffer
1183,1184d1183
< sun/nio/cs/StandardCharsets$Aliases
< sun/nio/cs/StandardCharsets$Cache
1187a1187,1190
> sun/nio/cs/ThreadLocalCoders
> sun/nio/cs/ThreadLocalCoders$1
> sun/nio/cs/ThreadLocalCoders$2
> sun/nio/cs/ThreadLocalCoders$Cache
1196a1200
> sun/nio/cs/UTF_8$Encoder
1231d1234
< sun/util/PreHashedMap

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

PR Comment: https://git.openjdk.org/jdk/pull/16971#issuecomment-1921462770