> Please review this change. Thanks!
> 
> The launcher test Settings.java compares command output against expected 
> text. The test was sensitive to the default locale because the expected 
> output was assumed to be in English.
> 
> This change makes the test run with the en-US locale explicitly to ensure 
> stable test results regardless of the default locale.
> 
> Testing:
> - jtreg test/jdk/tools/launcher/Settings.java with LANG=zh_CN.utf8 and 
> LC_ALL=zh_CN.utf8: passed.
> 
> ---------
> - [x] I confirm that I make this contribution in accordance with the [OpenJDK 
> Interim AI Policy](https://openjdk.org/legal/ai).

by1361092 has updated the pull request incrementally with one additional commit 
since the last revision:

  Address review comments

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

Changes:
  - all: https://git.openjdk.org/jdk/pull/32167/files
  - new: https://git.openjdk.org/jdk/pull/32167/files/8e1d7fbf..6b31687a

Webrevs:
 - full: https://webrevs.openjdk.org/?repo=jdk&pr=32167&range=02
 - incr: https://webrevs.openjdk.org/?repo=jdk&pr=32167&range=01-02

  Stats: 28 lines in 1 file changed: 0 ins; 1 del; 27 mod
  Patch: https://git.openjdk.org/jdk/pull/32167.diff
  Fetch: git fetch https://git.openjdk.org/jdk.git pull/32167/head:pull/32167

PR: https://git.openjdk.org/jdk/pull/32167

Reply via email to