build: Turn off default warnings if we find -Werror specified
authorColin Walters <walters@verbum.org>
Thu, 20 Jul 2017 19:37:27 +0000 (15:37 -0400)
committerAtomic Bot <atomic-devel@projectatomic.io>
Fri, 21 Jul 2017 20:45:47 +0000 (20:45 +0000)
Our CI runs use `-Werror`; there's no point to our default warning set kicking
in, it just bloats the command line output.

Closes: #1023
Approved by: jlebon

configure.ac

index 3fc0fabffa76d29e6219fb7788e856c54cb9b951..9728dee3da41630fa6922cac0fcd2f88cfacab87 100644 (file)
@@ -28,6 +28,7 @@ AC_SUBST([YEAR_VERSION], [year_version])
 AC_SUBST([RELEASE_VERSION], [release_version])
 AC_SUBST([PACKAGE_VERSION], [package_version])
 
+AS_IF([echo "$CFLAGS" | grep -q -E -e '-Werror($| )'], [], [
 CC_CHECK_FLAGS_APPEND([WARN_CFLAGS], [CFLAGS], [\
         -pipe \
         -Wall \
@@ -46,7 +47,7 @@ CC_CHECK_FLAGS_APPEND([WARN_CFLAGS], [CFLAGS], [\
         -Werror=misleading-indentation \
        -Werror=missing-include-dirs -Werror=aggregate-return \
   -Werror=unused-result \
-])
+])])
 AC_SUBST(WARN_CFLAGS)
 
 AC_MSG_CHECKING([for -fsanitize=address in CFLAGS])