fixes #114 #115

> **WARN** coveralls really sucks, I'm giving up and will find another provider
HerbertKoelman · Jun 8, 2019 · 7e8e7c0 · 7e8e7c0
1 parent 5f8d445
commit 7e8e7c0
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 4 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -31,7 +31,6 @@ script:
 
 # create coveralls.io report
 after_success:
-  - pwd
   - coveralls --root CMakeFiles/cpp-logger-static.dir/
 
 
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -10,9 +10,13 @@ project(
   DESCRIPTION "Simple C++ logger (${GIT_LOG})")
 
 option(GCOV "Activate GCOV options")
-if (GCOV AND ("${CMAKE_CXX_COMPILER_ID}" STREQUAL "GNU"))
-  message(STATUS "Setting GCOV compiler options")
-  add_compile_options(--coverage)
+if (GCOV )
+  if ("${CMAKE_CXX_COMPILER_ID}" STREQUAL "GNU")
+    message(STATUS "Setting GCOV compiler options")
+    add_compile_options(--coverage)
+  else()
+    message(SEND_ERROR "The GCOV option is only supported when using GNU...")
+  endif()
 endif()
 
 #find_package(Git CONFIG)