RFR: 8353489: Increase timeout and improve Windows compatibility in test/jdk/java/lang/ProcessBuilder/Basic.java [v2]

Oleksii Sylichenko duke at openjdk.org
Sat May 3 12:46:46 UTC 2025


On Sat, 3 May 2025 12:19:54 GMT, Eirik Bjørsnøs <eirbjo at openjdk.org> wrote:

>> Oleksii Sylichenko has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   8353489: replace `echo.` with `echo/`
>
> test/jdk/java/lang/ProcessBuilder/Basic.java line 1843:
> 
>> 1841:         //----------------------------------------------------------------
>> 1842:         try {
>> 1843:             String[] cmdp = Windows.is() ? new String[]{"cmd", "/c", "echo/"} : new String[]{"echo"};
> 
> A future maintainer without intimate familiarity with Windows commands and syntax would probably appreciate a short comment explaining the command chosen.

Would a comment with the following text be sufficient:

In Windows CMD (`cmd.exe`), `echo/` outputs a newline (i.e., an empty line).
Wrapping it with `cmd.exe /c` ensures compatibility in both native Windows and Cygwin environments.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/23933#discussion_r2072388153


More information about the core-libs-dev mailing list