On Tue, 31 Mar 2026 22:27:00 GMT, Ashay Rane <[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:/.

@raneashay 
Your change (at version 3fa99a94597ae59e4c52bf19fa1cf1e311f89020) is now ready 
to be sponsored by a Committer.

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

PR Comment: https://git.openjdk.org/jdk/pull/30523#issuecomment-4173384355

Reply via email to