Skip to content

Commit

Permalink
Don't override CMAKE_CXX_STANDARD set by parent projects
Browse files Browse the repository at this point in the history
When ADE used as a submodule, it overrides CMAKE_CXX_STANDARD set by parent project. E.g. parent projects defines CMAKE_CXX_STANDARD as 14 and during first cmake run it's applied, but during second cmake run ADE's value is used from cache.
No needs to keep CMAKE_CXX_STANDARD in cache, because it may vary from project to project.
  • Loading branch information
ilya-lavrenov committed Oct 10, 2023
1 parent ff027f8 commit 2589414
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion CMakeLists.txt
Expand Up @@ -18,7 +18,7 @@ option(FORCE_ADE_ASSERTS "Always enable ADE_ASSERT" OFF)
option(BUILD_ADE_DOCUMENTATION "Build doxygen documentation" OFF)
option(BUILD_WITH_STATIC_CRT "Build with static multi-threaded C Runtime (MS Windows/Visual Studio only)" OFF)

set(CMAKE_CXX_STANDARD 11 CACHE STRING "C++ standard")
set(CMAKE_CXX_STANDARD 11)

# TODO: this is horrible hack, we must follow cmake
# build/install policy
Expand Down

0 comments on commit 2589414

Please sign in to comment.