thodg/libgit2/cmake/Modules/EnableWarnings.cmake

• 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