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:/. the com/sun/di changes look good ------------- Marked as reviewed by lmesnik (Reviewer). PR Review: https://git.openjdk.org/jdk/pull/30523#pullrequestreview-4047425369
