On Wed, 1 Apr 2026 04:58:18 GMT, Yasumasa Suenaga <[email protected]> wrote:
>> Prior to this patch, both the build and the tests included several >> references to C:/. References in the build assumed that Windows is >> installed on C:/, which causes the build to fail when Windows is >> installed on a different drive. The references among tests assumed that >> the C:/ drive exists, which although mostly correct, is not guaranteed, >> making the tests fragile. >> >> This patch fixes the build references to use the `SYSTEMROOT` >> environment variable, which points to the Windows installation path, >> instead of hardcoded references to C:/Windows. This patch also updates >> tests to not use the presence of C:/ to detect Windows (instead relying >> on the output of `uname -s`) and to not assume that every Windows >> installation has a C:/. > > Can we build JDK on WSL after this change? > > BTW I'm not sure we can install Windows on not C... @YaSuenag My understanding is that it's possible to build the JDK on WSL even without this patch. This patch is for building using cygwin. I do have a machine where Windows is installed on D:, which is how I ran into the build problem. Hence this fix. ------------- PR Comment: https://git.openjdk.org/jdk/pull/30523#issuecomment-4170510228
