RFR: 8344275: tools/jpackage/windows/Win8301247Test.java fails on localized Windows platform [v3]

Alexey Semenyuk asemenyuk at openjdk.org
Wed Nov 20 03:19:15 UTC 2024


On Wed, 20 Nov 2024 01:47:34 GMT, Taizo Kurashige <duke at openjdk.org> wrote:

>> To resolve tools/jpackage/windows/Win8301247Test.java failure, I made "wmic" executed with "chcp 65001". This ensures that the encoding is UTF8 and that the English message "No Instance(s) Available." is output on localized windows platforms.
>> 
>> I have referred to the following for how to use chcp:
>> ・chcp | Microsoft Learn at https://learn.microsoft.com/en-us/windows-server/administration/windows-commands/chcp
>> ・Code Page Identifiers - Win32 apps | Microsoft Learn at https://learn.microsoft.com/en-us/windows/win32/intl/code-page-identifiers
>> 
>> After fix, I ran tools/jpackage tests importing jdk.jpackage.test.Executor or jdk.jpackage.test.WindowsHelper.killAppLauncherProcess on Windows Server 2019 (Japanese and English locales). I confirmed that they pass.
>> 
>> Thanks
>
> Taizo Kurashige has updated the pull request incrementally with one additional commit since the last revision:
> 
>   Fix indent

test/jdk/tools/jpackage/helpers/jdk/jpackage/test/Executor.java line 344:

> 342:     private List<String> prefixCommandLineArgs() {
> 343:         if (winEnglishOutput) {
> 344:             return List.of("cmd.exe", "/c", "chcp", "437", ">nul", "2>&1", "&&");

I assume `chcp 437` works as expected. The summary still references `chcp 65001` command.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/22142#discussion_r1849443613


More information about the core-libs-dev mailing list