Skip to content

Commit 5cf79dd

Browse files
Copilottsaubergine
andauthored
Update C++ standard to 17 required by Goby 3.x (if constexpr in zeromq headers)
Agent-Logs-Url: https://github.com/GobySoft/netsim/sessions/6895a37c-0532-4dd6-a4fa-7795a2fd1340 Co-authored-by: tsaubergine <732276+tsaubergine@users.noreply.github.com>
1 parent 09cb626 commit 5cf79dd

1 file changed

Lines changed: 6 additions & 3 deletions

File tree

CMakeLists.txt

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,12 @@ set(CMAKE_MODULE_PATH "${CMAKE_SOURCE_DIR}/cmake_modules/")
44

55
cmake_minimum_required(VERSION 3.5.1 FATAL_ERROR)
66

7-
# require C++14
8-
set(CMAKE_CXX_STANDARD 14)
9-
set(CMAKE_CXX_STANDARD_REQUIRED ON)
7+
# require C++17 (Goby 3.x headers use C++17 features such as if constexpr)
8+
if(NOT CMAKE_CXX_STANDARD OR CMAKE_CXX_STANDARD LESS 17)
9+
set(CMAKE_CXX_STANDARD 17)
10+
set(CMAKE_CXX_STANDARD_REQUIRED ON)
11+
message(STATUS "Setting CMAKE_CXX_STANDARD to 17 as required by Goby 3.x")
12+
endif()
1013

1114
# versions
1215
set(PROJECT_VERSION_MAJOR "1")

0 commit comments

Comments
 (0)