Skip to content

Commit a78686d

Browse files
committed
Disable gcov by default
1 parent 6604cac commit a78686d

2 files changed

Lines changed: 10 additions & 5 deletions

File tree

.github/workflows/sanitizers.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -51,8 +51,8 @@ jobs:
5151
-DCSV_CXX_STANDARD=${{ matrix.cxx_standard }} \
5252
-DCMAKE_CXX_COMPILER=g++ \
5353
-DCMAKE_C_COMPILER=gcc \
54-
-DCMAKE_CXX_FLAGS="${{ matrix.flag }} -g -fno-profile-arcs -fno-test-coverage" \
55-
-DCMAKE_C_FLAGS="${{ matrix.flag }} -g -fno-profile-arcs -fno-test-coverage"
54+
-DCMAKE_CXX_FLAGS="${{ matrix.flag }} -g" \
55+
-DCMAKE_C_FLAGS="${{ matrix.flag }} -g"
5656
5757
- name: Build with ${{ matrix.name }}
5858
run: cmake --build build --config Debug
@@ -92,8 +92,8 @@ jobs:
9292
-DCSV_CXX_STANDARD=17 \
9393
-DCMAKE_CXX_COMPILER=g++ \
9494
-DCMAKE_C_COMPILER=gcc \
95-
-DCMAKE_CXX_FLAGS="-g -O1 -fno-profile-arcs -fno-test-coverage" \
96-
-DCMAKE_C_FLAGS="-g -O1 -fno-profile-arcs -fno-test-coverage"
95+
-DCMAKE_CXX_FLAGS="-g -O1" \
96+
-DCMAKE_C_FLAGS="-g -O1"
9797
9898
- name: Build
9999
run: cmake --build build --config Debug

CMakeLists.txt

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,7 @@ else()
88
endif(CSV_CXX_STANDARD)
99

1010
option(BUILD_PYTHON "Build Python Binding" OFF)
11+
option(ENABLE_CODE_COVERAGE "Enable code coverage instrumentation" OFF)
1112

1213
message("Building CSV library using C++${CMAKE_CXX_STANDARD}")
1314

@@ -29,7 +30,11 @@ if(MSVC)
2930
else()
3031
# Ignore Visual Studio pragma regions
3132
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} -Wno-unknown-pragmas")
32-
set(CMAKE_CXX_FLAGS_DEBUG "${CMAKE_CXX_FLAGS_DEBUG} --coverage -Og")
33+
34+
if(ENABLE_CODE_COVERAGE)
35+
message("Code coverage instrumentation enabled")
36+
set(CMAKE_CXX_FLAGS_DEBUG "${CMAKE_CXX_FLAGS_DEBUG} --coverage -Og")
37+
endif()
3338
endif(MSVC)
3439

3540
set(CSV_ROOT_DIR ${CMAKE_CURRENT_LIST_DIR})

0 commit comments

Comments
 (0)