[jdk11u-dev] RFR: 8360816: [11u] Use default value for ProgramFiles(x86) in GHA [v7]
Severin Gehwolf
sgehwolf at openjdk.org
Wed Jul 2 09:40:43 UTC 2025
On Wed, 2 Jul 2025 08:57:35 GMT, Antonio Vieiro <duke at openjdk.org> wrote:
>> A fix for [JDK-8360816](https://bugs.openjdk.org/browse/JDK-8360816).
>>
>> The `ProgramFiles(x86)` environment variable is not properly propagated to JTReg tests in JDK11, which makes the AOT compiler tests fail on the new `windows-2025` GHA runners, as manifested in the backport of [https://bugs.openjdk.org/browse/JDK-8358538](JDK-8358538).
>>
>> Propagating this environment variable to JTRegs may require creating devkits for Windows 2022, but previous attempts were discarded [JDK-8283723](https://bugs.openjdk.org/browse/JDK-8283723).
>>
>> This fix prints a warning message when this environment variable is not set, and then uses a default value (`C:\Program Files (x86)`) that makes it possible to run the AOT tests with these new `windows-2025` GHA runners. If the `ProgramFiles(x86)` env variable is set the fix has no effect, ensuring the previous behaviour. If the proposed default value is incorrect the AOT tests will continue to fail as before.
>>
>> The PR is on top of https://github.com/openjdk/jdk11u-dev/pull/3052 to verify the fix works as intended with the new `windows-2025` GHA runners.
>
> Antonio Vieiro has updated the pull request incrementally with one additional commit since the last revision:
>
> Set JTReg ProgramFiles(x86) env. variable on Windows
Seems OK.
-------------
Marked as reviewed by sgehwolf (Reviewer).
PR Review: https://git.openjdk.org/jdk11u-dev/pull/3056#pullrequestreview-2978467217
More information about the jdk-updates-dev
mailing list