[cmake] Better way to enforce C++ 17 in CMakeLists.txt

olivier-stasse · olivier-stasse · commit 9d328dee6747 · 2023-10-12T00:43:41.000+02:00
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -31,9 +31,7 @@ include(cmake/ros.cmake)
 compute_project_args(PROJECT_ARGS LANGUAGES CXX)
 project(${PROJECT_NAME} ${PROJECT_ARGS})
 
-# if(NOT CMAKE_CXX_STANDARD)
-set(CMAKE_CXX_STANDARD 17)
-# endif()
+check_minimal_cxx_standard(17 ENFORCE)
 
 cmake_policy(SET CMP0057 NEW)
 find_package(ament_cmake REQUIRED)
