EnableWarnings.cmake

Tag

Show log
Commit

Hash : 6af6170b
Author :
Date : 2019-01-05T20:50:42

Make ENABLE_WERROR actually work

This change explicitly adds -Werror to the CFLAGS.

Due to the way that the ADD_C_FLAG_IF_SUPPORTED() macro was mangling the
flag name to convert it into a define name, any warning that had a dash
in its name was not being correctly enabled. Additionally, any flag that
is enabled implicitly by the compiler (like -Wunused-result and
-Wdeprecated-declarations) would not cause an error unless they were
explicitly enabled with the ENABLE_WARNINGS() macro.

Download

cmake/Modules/EnableWarnings.cmake

MACRO(ENABLE_WARNINGS flag)
	ADD_C_FLAG_IF_SUPPORTED(-W${flag})
ENDMACRO()

MACRO(DISABLE_WARNINGS flag)
	ADD_C_FLAG_IF_SUPPORTED(-Wno-${flag})
ENDMACRO()

IF(ENABLE_WERROR)
	ADD_C_FLAG_IF_SUPPORTED(-Werror)
ENDIF()

Download

thodg/libgit2/cmake/Modules/EnableWarnings.cmake

Commit

cmake/Modules/EnableWarnings.cmake