RFR: 8306543: GHA: MSVC installation is failing when component is already installed

Aleksey Shipilev shade at openjdk.org
Thu Apr 20 17:27:43 UTC 2023


On Thu, 20 Apr 2023 16:23:26 GMT, Christoph Langer <clanger at openjdk.org> wrote:

>> Currrent GHA fails on MSVC installation. Looks like the required versions are already installed in the runner image, and this is why the command fails. Our configure scripts look for the specific minor version, and the builds still pass if I drop the VC installation block completely. This tells me the appropriate minor version is already installed. 
>> 
>> For safety, I think we can just ignore the errors from the MSVC installation. In case the runner images moves on to another minor version later, the dependencies would start to install successfully again.
>> 
>> See the GHA run. Both Windows x86_64 and Windows AArch64 select `1429~1.301`:
>> 
>> 
>> * Toolchain:      microsoft (Microsoft Visual Studio 2019)
>> * C Compiler:     Version 19.29.30148 (at /c/progra~2/micros~2/2019/enterp~1/vc/tools/msvc/1429~1.301/bin/hostx64/x64/cl.exe)
>> * C++ Compiler:   Version 19.29.30148 (at /c/progra~2/micros~2/2019/enterp~1/vc/tools/msvc/1429~1.301/bin/hostx64/x64/cl.exe)
>> 
>> 
>> The last successful GHA run I had selected the same versions:
>> 
>> 
>> Tools summary:
>> * Toolchain:      microsoft (Microsoft Visual Studio 2019)
>> * C Compiler:     Version 19.29.30148 (at /c/progra~2/micros~2/2019/enterp~1/vc/tools/msvc/1429~1.301/bin/hostx64/x64/cl.exe)
>> * C++ Compiler:   Version 19.29.30148 (at /c/progra~2/micros~2/2019/enterp~1/vc/tools/msvc/1429~1.301/bin/hostx64/x64/cl.exe)
>> 
>> 
>> Additional testing:
>>  - [x] Ad-hoc GHA runs pass the configure/initial-build
>>  - [x] Full GHA runs pass (this PR)
>
> Looks good. Too bad that the Windows installer is not showing any sensible output.
> We will want to backport this to 11u and 17u soon, too.

Thanks for review, @RealCLanger!
Any other reviews, please, so I can push?

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

PR Comment: https://git.openjdk.org/jdk/pull/13558#issuecomment-1516691879



More information about the build-dev mailing list