[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