Skip to content

Commit bb706f5

Browse files
authored
build: switch from c90 to c11 (#4743)
After 14 years that should be fairly safe, right? Right!? Not safe enough for Windows Server 2016 apparently; there are build errors coming from system headers. The GHA images are slated for removal in a month anyway so upgrade them to Windows Server 2025. Fixes: #4742
1 parent 2b96e47 commit bb706f5

4 files changed

Lines changed: 4 additions & 5 deletions

File tree

.github/workflows/CI-win.yml

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,13 +21,12 @@ jobs:
2121
fail-fast: false
2222
matrix:
2323
config:
24-
- {toolchain: Visual Studio 16 2019, arch: Win32, server: 2019}
25-
- {toolchain: Visual Studio 16 2019, arch: x64, server: 2019}
2624
- {toolchain: Visual Studio 17 2022, arch: Win32, server: 2022}
2725
- {toolchain: Visual Studio 17 2022, arch: x64, server: 2022}
2826
- {toolchain: Visual Studio 17 2022, arch: x64, server: 2022, config: ASAN}
2927
- {toolchain: Visual Studio 17 2022, arch: x64, server: 2022, config: UBSAN}
3028
- {toolchain: Visual Studio 17 2022, arch: arm64, server: 2022}
29+
- {toolchain: Visual Studio 17 2022, arch: x64, server: 2025}
3130
steps:
3231
- uses: actions/checkout@v4
3332
- name: Build

CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ include(CTest)
2020
set(CMAKE_C_VISIBILITY_PRESET hidden)
2121
set(CMAKE_C_STANDARD_REQUIRED ON)
2222
set(CMAKE_C_EXTENSIONS ON)
23-
set(CMAKE_C_STANDARD 90)
23+
set(CMAKE_C_STANDARD 11)
2424

2525
set(CMAKE_EXPORT_COMPILE_COMMANDS ON)
2626

SUPPORTED_PLATFORMS.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
|---|---|---|---|
55
| GNU/Linux | Tier 1 | Linux >= 3.10 with glibc >= 2.17 | |
66
| macOS | Tier 1 | macOS >= 11 | Currently supported macOS releases |
7-
| Windows | Tier 1 | >= Windows 10 | VS 2015 and later are supported |
7+
| Windows | Tier 1 | >= Windows 10 | VS 2017 and later are supported |
88
| FreeBSD | Tier 2 | >= 12 | |
99
| AIX | Tier 2 | >= 6 | Maintainers: @libuv/aix |
1010
| IBM i | Tier 2 | >= IBM i 7.2 | Maintainers: @libuv/ibmi |

configure.ac

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -33,7 +33,7 @@ CC_ATTRIBUTE_VISIBILITY([default], [
3333
# we exclude -fno-strict-aliasing for xlc
3434
CC_CHECK_FLAG_SUPPORTED_APPEND([-fno-strict-aliasing])
3535
CC_CHECK_CFLAGS_APPEND([-g])
36-
CC_CHECK_CFLAGS_APPEND([-std=gnu89])
36+
CC_CHECK_CFLAGS_APPEND([-std=gnu11])
3737
CC_CHECK_CFLAGS_APPEND([-Wall])
3838
CC_CHECK_CFLAGS_APPEND([-Wextra])
3939
CC_CHECK_CFLAGS_APPEND([-Wno-long-long])

0 commit comments

Comments
 (0)