Skip to content

Commit 4f2ff41

Browse files
hartworkvitalybuka
authored andcommitted
Move troublesome default -Werror from CMake to CI
Signed-off-by: Sebastian Pipping <[email protected]>
1 parent f9466eb commit 4f2ff41

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

.github/workflows/cmake-multi-platform.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,9 @@ jobs:
5252
run: >
5353
cmake -B ${{ steps.strings.outputs.build-output-dir }}
5454
-DCMAKE_CXX_COMPILER=${{ matrix.cpp_compiler }}
55+
-DCMAKE_CXX_FLAGS='-Werror'
5556
-DCMAKE_C_COMPILER=${{ matrix.c_compiler }}
57+
-DCMAKE_C_FLAGS='-Werror'
5658
-DCMAKE_BUILD_TYPE=${{ matrix.build_type }}
5759
-DLIB_PROTO_MUTATOR_DOWNLOAD_PROTOBUF=ON
5860
-GNinja

CMakeLists.txt

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -80,7 +80,7 @@ unset(CMAKE_REQUIRED_FLAGS)
8080
check_cxx_compiler_flag(-Wstring-conversion LIB_PROTO_MUTATOR_HAS_WSTRING_CONVERSION)
8181

8282
if (NOT MSVC)
83-
set(EXTRA_FLAGS "-fno-exceptions -Werror -Wall")
83+
set(EXTRA_FLAGS "-fno-exceptions -Wall")
8484
if (LIB_PROTO_MUTATOR_HAS_WSTRING_CONVERSION)
8585
set(EXTRA_FLAGS "${EXTRA_FLAGS} -Wstring-conversion")
8686
endif()

0 commit comments

Comments
 (0)