From 22700faf88bdf7be0bbd8dad658f0d08816a7c51 Mon Sep 17 00:00:00 2001 From: Philip Withnall Date: Thu, 3 Aug 2017 11:55:23 +0100 Subject: [PATCH] build: Loosen --enable-compile-warnings check MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Instead of requiring --enable-compile-warnings or --enable-compile-warnings=yes, allow any value which is not ‘no’. This enables compile warnings for --enable-compile-warnings=maximum or --enable-compile-warnings=error, which are common values for other GNOME projects. While we don’t change our behaviour for [yes, maximum, error], at least it means the warnings are enabled now, rather than disabled. Signed-off-by: Philip Withnall --- configure.ac | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/configure.ac b/configure.ac index fe9438f52..5c2bf8e6b 100644 --- a/configure.ac +++ b/configure.ac @@ -3390,7 +3390,7 @@ AC_ARG_ENABLE(compile-warnings, [AS_HELP_STRING([--disable-compile-warnings], [Don't use builtin compiler warnings])],, enable_compile_warnings=yes) -AS_IF([test "x$enable_compile_warnings" = xyes], [ +AS_IF([test "x$enable_compile_warnings" != xno], [ CC_CHECK_FLAGS_APPEND([GLIB_WARN_CFLAGS], [CFLAGS], [\ -Wall -Wstrict-prototypes -Werror=declaration-after-statement \ -Werror=missing-prototypes -Werror=implicit-function-declaration \