[jdk8u-dev] Integrated: 8276841: Add support for Visual Studio 2022

Alexander Scherbatiy alexsch at openjdk.org
Thu Apr 20 12:37:52 UTC 2023


On Fri, 31 Mar 2023 15:00:04 GMT, Alexander Scherbatiy <alexsch at openjdk.org> wrote:

> This is a backport of the fix from jdk11u-dev repository: https://github.com/openjdk/jdk11u-dev/commit/0ba5a81e7c0117855665a28a43f9589635e94222
> The fix is a parity fix for JDK-8298996 in 8u371 from Oracle.
> 
> The fix is manually merged with jdk8u-dev because it has conflicts in lines before the fix:
> jdk11u-dev: `VALID_VS_VERSIONS="2017 2019 2013 2015 2012 2010"`
> jdk8u-dev: `VALID_VS_VERSIONS="2010 2012 2013 2015 2017 2019"`
> The conflict is resolved so `2022` year is placed after `2019` in jdk8u-dev.
> 
> The other part is backported as is into jdk8u-dev. The `generated-configure.sh` file is updated as well.
> 
> To test the fix the jdk8u-dev was built on Windows Server 2012 R2 with:
> 
> * Toolchain:      microsoft (Microsoft Visual Studio 2022 17.5.2 (devkit))
> * C Compiler:     Version 19.31.31107 (at /cygdrive/c/cygwin64/VS2022-17.5.2-devkit/VC/bin/x64/cl)
> * C++ Compiler:   Version 19.31.31107 (at /cygdrive/c/cygwin64/VS2022-17.5.2-devkit/VC/bin/x64/cl)
> 
> and hotspot and compact3 tests were run. There were only 3 new failed tests comparing to the build without the fix.
>  ```
> jdk/internal/platform/cgroup/TestCgroupMetrics.java.TestCgroupMetrics
> jdk/internal/platform/docker/TestDockerMemoryMetrics.java.TestDockerMemoryMetrics
> jdk/internal/platform/docker/TestSystemMetrics.java.TestSystemMetrics
> 
> The first test passes after the second run. 
> I rerun the tests with build without the fix and these 3 tests failed as well.
> It looks like that these failures actually are not related to the fix.

This pull request has now been integrated.

Changeset: 644e79d9
Author:    Alexander Scherbatiy <alexsch at openjdk.org>
URL:       https://git.openjdk.org/jdk8u-dev/commit/644e79d9440ea7ba08f25b762d3d739e511c8f40
Stats:     32 lines in 2 files changed: 30 ins; 0 del; 2 mod

8276841: Add support for Visual Studio 2022

Reviewed-by: sgehwolf
Backport-of: 0ba5a81e7c0117855665a28a43f9589635e94222

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

PR: https://git.openjdk.org/jdk8u-dev/pull/295


More information about the jdk8u-dev mailing list