Skip to content

follow-up for #13050 - fixed CMake warning when BUILD_TESTS is not actually provided #8277

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
firewave wants to merge 1 commit into
base: main
Choose a base branch
from
+2 −1
Open

follow-up for #13050 - fixed CMake warning when BUILD_TESTS is not actually provided #8277

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 2 additions & 1 deletion cmake/options.cmake
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -56,7 +56,7 @@ option(BUILD_CORE_DLL "Build lib as cppcheck-core.dll with Visual Studio"
if(BUILD_CORE_DLL AND NOT MSVC)
message(FATAL_ERROR "Building of lib as DLL is only supported with Visual Studio")
endif()
option(BUILD_TESTS "Build tests" OFF)
# need to check before the option() specifying it or it will be defined
if(DEFINED BUILD_TESTS)
message(WARNING "BUILD_TESTS has been deprecated and will be removed in Cppcheck 2.22 - please use BUILD_TESTING instead")
if(DEFINED BUILD_TESTING)
Expand All @@ -68,6 +68,7 @@ elseif(NOT DEFINED BUILD_TESTING)
# disable tests by default - TODO: remove this
set(BUILD_TESTING OFF)
endif()
option(BUILD_TESTS "Build tests" OFF)
option(REGISTER_TESTS "Register tests in CTest" ON)
option(ENABLE_CHECK_INTERNAL "Enable internal checks" OFF)
option(DISABLE_DMAKE "Disable run-dmake dependencies" OFF)
Expand Down
Loading