> 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 refreshed the contents of this pull request, and previous commits 
have been removed. The incremental views will show differences compared to the 
previous content of the PR. The pull request contains one new commit since the 
last revision:

  8388366: Launcher test stability regardless of the default locale

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

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

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

  Stats: 16 lines in 1 file changed: 9 ins; 3 del; 4 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