RFR: 8346719: Add relaunchers to the static JDK image for missing executables [v9]

Magnus Ihse Bursie ihse at openjdk.org
Wed Sep 17 14:54:21 UTC 2025


On Mon, 15 Sep 2025 17:29:59 GMT, Roger Riggs <rriggs at openjdk.org> wrote:

>> Magnus Ihse Bursie has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Remove debug code
>
> src/java.base/windows/native/launcher/relauncher.c line 153:
> 
>> 151:     }
>> 152: 
>> 153:     // Our executable name (should not be quoted)
> 
> ok here, because the full path is being explicitly passed to CreateProcess.
> If the command line is to be parsed and the executable path contains a space, CreateProcess does say it should be quoted.  It would be good to have a test case with a space in the executable path.

Indeed, this seems sus. I need to dig more into this. I recall initially quoting all arguments and then running into problems with the executable. Maybe it is that spaces needs to be quoted, but not other special characters. I'll check what you are doing in ProcessImpl.java and see if I can align my code to do the same.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24380#discussion_r2355810340


More information about the core-libs-dev mailing list