On Wed, 4 Jun 2025 16:55:12 GMT, Magnus Ihse Bursie <i...@openjdk.org> wrote:

>> .github/workflows/main.yml line 313:
>> 
>>> 311:     with:
>>> 312:       platform: windows-x64
>>> 313:       msvc-toolset-version: '14.43'
>> 
>> I wish I understood why `14.43`, since I don't see any reference to it in 
>> current runner docs: 
>> https://github.com/actions/runner-images/blob/main/images/windows/Windows2025-Readme.md.
>>  But I see you have tried other versions, and this one works?
>> 
>> From https://learn.microsoft.com/en-us/lifecycle/faq/visual-c-faq it sounds 
>> like `14.43` is not supported/final one, listed as `TBD` for start/end 
>> dates. (Compare with `14.29` for VS 2019.) `14.42` is listed as supported 
>> until July 14, 2026. I see you have tried `14.42`: 
>> https://github.com/openjdk/jdk/pull/25628/commits/6f7922f5c70e4ffef82005b3ea702673b589eb64
>>  -- did it not work?
>
> Yes, neither 14.44 nor 14.42, which I thought I had good reasons to believe 
> should work did in fact work. :-( I'm not very keen on trying to debug this 
> further. The brownouts have already started. Feel free to either take over 
> this PR, or come up with a follow-up where you try to figure out what VS 
> versions work...

Yeah, all right, just checking. If `14.42` worked, it would be tempting to use 
that. But it does not. So if `14.43` works, let's ship that. We can amend it 
later. Fix the test job definition: 
https://github.com/openjdk/jdk/pull/25628#issuecomment-2939972202 -- and I'll 
approve.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/25628#discussion_r2127051108

Reply via email to