Better coverage dependencies

PJK · Apr 26, 2015 · 3bae447 · 3bae447
1 parent 3f60e9b
commit 3bae447
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -20,7 +20,6 @@ add_custom_target(coverage
                   COMMAND genhtml coverage.info --highlight --legend --output-directory coverage_html
                   COMMAND ${CMAKE_COMMAND} -E remove ${CMAKE_CURRENT_SOURCE_DIR}/test/stream_expectations.c
                   COMMAND echo "Coverage report ready: file://${CMAKE_CURRENT_BINARY_DIR}/coverage_html/index.html")
-
 include_directories(src)
 
 
@@ -32,7 +31,7 @@ endif(CUSTOM_ALLOC)
 option(COVERAGE "Enable code coverage instrumentation" OFF)
 if(COVERAGE)
     message("Configuring code coverage instrumentation")
-    if(NOT CMAKE_C_COMPILER EQUAL "gcc")
+    if(NOT CMAKE_C_COMPILER MATCHES "gcc")
         message(WARNING "Gcov instrumentation only works with GCC")
     endif()
     # https://gcc.gnu.org/onlinedocs/gcc/Debugging-Options.html

diff --git a/test/CMakeLists.txt b/test/CMakeLists.txt
@@ -15,6 +15,7 @@ foreach(TEST ${TESTS})
   target_link_libraries(${NAME} cmocka)
   target_link_libraries(${NAME} cbor)
   add_test(${NAME} ${NAME})
+  add_dependencies(coverage ${NAME})
 
   add_custom_command(TARGET ${NAME} POST_BUILD
       COMMAND ${CMAKE_COMMAND} -E remove ${CMAKE_CURRENT_SOURCE_DIR}/stream_expectations.c