Skip to content

Commit 1ba55a2

Browse files
committed
cmake: use cmake_dependent_option for WITH_BREAKPAD
a bit simpler without the WITH_BREAKPAD_DEFAULT part, and causes the WITH_BREAKPAD option to be hidden from cmake-gui on WIN32 Signed-off-by: Casey Bodley <[email protected]>
1 parent 8b8c88a commit 1ba55a2

File tree

1 file changed

+3
-7
lines changed

1 file changed

+3
-7
lines changed

CMakeLists.txt

Lines changed: 3 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -538,13 +538,9 @@ if(WITH_CATCH2)
538538
message("-- Enabled Catch2 support")
539539
endif()
540540

541-
if(WIN32)
542-
set(WITH_BREAKPAD_DEFAULT OFF)
543-
else()
544-
set(WITH_BREAKPAD_DEFAULT ON)
545-
endif()
546-
547-
option(WITH_BREAKPAD "Build with Google Breakpad crash reporter" ${WITH_BREAKPAD_DEFAULT})
541+
# enable breakpad unless win32
542+
CMAKE_DEPENDENT_OPTION(WITH_BREAKPAD "Build with Google Breakpad crash reporter" OFF
543+
"WIN32" ON)
548544
if(WITH_BREAKPAD)
549545
set(HAVE_BREAKPAD ON)
550546
message("-- Enabled Google Breakpad crash reporter")

0 commit comments

Comments
 (0)