Skip to content

Commit 4a07ee9

Browse files
authored
Use cross-platform way to set -Werror (#351)
1 parent c143f37 commit 4a07ee9

File tree

1 file changed

+3
-3
lines changed

1 file changed

+3
-3
lines changed

cmake/configure.cmake

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,8 @@ else ()
2424
set( CMAKE_CXX_STANDARD 20 )
2525
endif ()
2626

27+
set(CMAKE_COMPILE_WARNING_AS_ERROR ON)
28+
2729
if( UNIX )
2830
set(COMMON_COMPILER_FLAGS
2931
"${COMMON_COMPILER_FLAGS} \
@@ -54,8 +56,6 @@ if( UNIX )
5456
set (COMMON_COMPILER_FLAGS "${COMMON_COMPILER_FLAGS} -Wno-cast-align")
5557
endif()
5658

57-
set (COMMON_COMPILER_FLAGS "${COMMON_COMPILER_FLAGS} -Werror")
58-
5959
set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${COMMON_COMPILER_FLAGS}")
6060
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${COMMON_COMPILER_FLAGS}")
6161
if (USE_COVERAGE)
@@ -73,7 +73,7 @@ if( MSVC )
7373
set(CMAKE_CXX_FLAGS "${CMAKE_CXX_FLAGS} ${COMMON_COMPILER_FLAGS}")
7474
endif()
7575
set(CMAKE_EXE_LINKER_FLAGS "${CMAKE_EXE_LINKER_FLAGS}")
76-
set(COMMON_FLAGS "/W4 /wd4267 /wd4244 /wd4100 /WX")
76+
set(COMMON_FLAGS "/W4 /wd4267 /wd4244 /wd4100")
7777
set(CMAKE_C_FLAGS_RELEASE "${CMAKE_C_FLAGS_RELEASE} ${COMMON_FLAGS}" )
7878
set(CMAKE_C_FLAGS_DEBUG "${CMAKE_C_FLAGS_DEBUG} ${COMMON_FLAGS}")
7979
set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} ${COMMON_FLAGS}" )

0 commit comments

Comments
 (0)