On Thu, 20 Apr 2023 11:56:37 GMT, Aleksey Shipilev <sh...@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.

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

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

Reply via email to