Skip to content

Commit

Permalink
cmake: Treat warnings as errors on Clang and GCC
Browse files Browse the repository at this point in the history
-Wformat-security is enabled, some Linux packages build system have it
enabled.
obsproject#5766

Also ensure that null conversion warning are enabled (C++ only for GCC)
because its enablement by default can depend on the CMake generator.
  • Loading branch information
tytan652 committed Jul 21, 2022
1 parent ff4f4fc commit 6fa05e8
Showing 1 changed file with 6 additions and 2 deletions.
8 changes: 6 additions & 2 deletions cmake/Modules/CompilerConfig.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -84,16 +84,20 @@ else()
endif()

add_compile_options(
-Werror
-Wextra
-Wvla
-Wformat
-Wformat-security
-Wno-unused-function
-Wno-missing-field-initializers
-fno-strict-aliasing
"$<$<COMPILE_LANGUAGE:C>:-Werror-implicit-function-declaration;-Wno-missing-braces>"
"$<$<BOOL:${USE_LIBCXX}>:-stdlib=libc++>"
"$<$<CONFIG:DEBUG>:-DDEBUG=1;-D_DEBUG=1>"
"$<$<COMPILE_LANG_AND_ID:CXX,AppleClang,Clang>:-fcolor-diagnostics>"
"$<$<COMPILE_LANG_AND_ID:C,AppleClang,Clang>:-fcolor-diagnostics>")
"$<$<COMPILE_LANG_AND_ID:CXX,AppleClang,Clang>:-Wnull-conversion;-fcolor-diagnostics>"
"$<$<COMPILE_LANG_AND_ID:C,AppleClang,Clang>:-Wnull-conversion;-fcolor-diagnostics>"
"$<$<COMPILE_LANG_AND_ID:CXX,GNU>:-Wconversion-null>")

if(OBS_CODESIGN_LINKER)
add_link_options("LINKER:$<$<PLATFORM_ID:Darwin>:-adhoc_codesign>")
Expand Down

0 comments on commit 6fa05e8

Please sign in to comment.