Skip to content

Commit 10f444f

Browse files
authored
build: Default to C++17 mode (#1818)
This does not change the minimum C++ standard, which is still C++14, even in main. But it does change the default in main to build in C++17 mode, and to get 14 you have to say so explicitly. We do anticipate bumping the minimum before this fall's release, and this is just a half step to get us ready. Maybe it will help shake loose any compatibility problems and remind people on main that they are using a branch that in a few months will have a new minimum C++. This will not be backported to the release. Signed-off-by: Larry Gritz <lg@larrygritz.com>
1 parent 65024b0 commit 10f444f

File tree

1 file changed

+1
-1
lines changed

1 file changed

+1
-1
lines changed

src/cmake/compiler.cmake

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,7 @@ message (STATUS "CMAKE_CXX_COMPILER_ID = ${CMAKE_CXX_COMPILER_ID}")
2525
###########################################################################
2626
# C++ language standard
2727
#
28-
set (CMAKE_CXX_STANDARD 14 CACHE STRING
28+
set (CMAKE_CXX_STANDARD 17 CACHE STRING
2929
"C++ standard to build with (14, 17, 20, etc.)")
3030
set (DOWNSTREAM_CXX_STANDARD 14 CACHE STRING
3131
"C++ minimum standard to impose on downstream clients")

0 commit comments

Comments
 (0)