diff options
author | David Benjamin <davidben@google.com> | 2023-05-05 15:53:48 -0400 |
---|---|---|
committer | Boringssl LUCI CQ <boringssl-scoped@luci-project-accounts.iam.gserviceaccount.com> | 2023-05-08 21:46:15 +0000 |
commit | 2aae3f58b42e75690f28853f712a2e204857b7f6 (patch) | |
tree | fc9e64c9a85c3fae91a849f571d13c41e25192eb /CMakeLists.txt | |
parent | a972b78d1b11009cd07852fb4be2cc938489e031 (diff) | |
download | boringssl-2aae3f58b42e75690f28853f712a2e204857b7f6.zip boringssl-2aae3f58b42e75690f28853f712a2e204857b7f6.tar.gz boringssl-2aae3f58b42e75690f28853f712a2e204857b7f6.tar.bz2 |
Bump the minimum supported MSVC version to VS2019
This aligns with https://github.com/google/oss-policies-info/pull/8 and
https://github.com/grpc/grpc/pull/32614. VS2019 adds a C11 mode, which
is useful for us, because it means stdalign.h works correctly.
Also bump the minimum Windows SDK to
https://devblogs.microsoft.com/cppblog/c11-and-c17-standard-support-arriving-in-msvc/.
If you have a new MSVC, CMake will enable C11 mode by default. But if
C11 mode is enabled but your Windows SDK is too old, things break.
After this change, the CI will include some redundant configurations.
All the VS2017 configurations will start testing on VS2019, so the
VS2019-specific configurations won't do anything. I'll follow this up
with a change to bump those to VS2022, where we're currently missing
coverage.
Update-Note: BoringSSL now requires VS2019 or later and no longer
supports VS2017. VS2017 has been past its "mainstream end date" for over
a year now, per
https://learn.microsoft.com/en-us/lifecycle/products/visual-studio-2017
Change-Id: I3f359e8ea7c9428ddaa9fcc4ffead2ef903398be
Reviewed-on: https://boringssl-review.googlesource.com/c/boringssl/+/59665
Reviewed-by: Adam Langley <agl@google.com>
Commit-Queue: David Benjamin <davidben@google.com>
Diffstat (limited to 'CMakeLists.txt')
-rw-r--r-- | CMakeLists.txt | 5 |
1 files changed, 0 insertions, 5 deletions
diff --git a/CMakeLists.txt b/CMakeLists.txt index aac5f0d..c939e42 100644 --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -203,11 +203,6 @@ elseif(MSVC) set(MSVC_DISABLED_WARNINGS_LIST "C4100" # 'exarg' : unreferenced formal parameter "C4127" # conditional expression is constant - # C4204 and C4221 are C89-only restrictions which were dropped in C99, but - # VS2017 warns about it. They can be removed when we require VS2019. - "C4204" # nonstandard extension used: non-constant aggregate initializer - "C4221" # nonstandard extension used : 'identifier' : cannot be - # initialized using address of automatic variable "C4244" # 'function' : conversion from 'int' to 'uint8_t', # possible loss of data "C4267" # conversion from 'size_t' to 'int', possible loss of data |