fix: make cmake sanitizer inclusion optional as it breaks the feature…

…_summary function

Hopefully this will be fixed in arsenm/sanitizers-cmake#18

CMakeLists.txt

@@ -32,6 +32,8 @@ execute_process(
     OUTPUT_STRIP_TRAILING_WHITESPACE
 )
 
+option(ENABLE_SANITIZERS "Enable builds using sanitizers" off)
+
 # include external libraries
 add_subdirectory(lib)
 

src/CMakeLists.txt

@@ -2,10 +2,6 @@ cmake_minimum_required(VERSION 3.2)
 
 set(CMAKE_POSITION_INDEPENDENT_CODE true)
 
-# add support for sanitizers
-set(CMAKE_MODULE_PATH "${PROJECT_SOURCE_DIR}/lib/sanitizers-cmake/cmake" ${CMAKE_MODULE_PATH})
-find_package(Sanitizers REQUIRED)
-
 # required to link to correct thread library
 find_package(Threads REQUIRED)
 find_package(X11 REQUIRED)

src/qt-ui/CMakeLists.txt

@@ -37,7 +37,14 @@ target_compile_definitions(libappimageupdate-qt_static PRIVATE -DQT_UI)
 
 # Qt GUI application
 add_executable(AppImageUpdate main.cpp)
-add_sanitizers(AppImageUpdate)
+
+if (ENABLE_SANITIZERS)
+    # add support for sanitizers.  Partially broken: fix at https://github.com/arsenm/sanitizers-cmake/pull/18
+    set(CMAKE_MODULE_PATH "${PROJECT_SOURCE_DIR}/lib/sanitizers-cmake/cmake" ${CMAKE_MODULE_PATH})
+    find_package(Sanitizers REQUIRED)
+
+    add_sanitizers(AppImageUpdate)
+endif (ENABLE_SANITIZERS)
 
 # link libraries
 target_link_libraries(AppImageUpdate libappimageupdate libappimageupdate-qt)