colin/ModemManager
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

m4/compiler_warnings.m4: Let -Werror be disabled

Browse Source 
It's annoying for distributors to build with -Werror, since it means
that compiler upgrades can break the build. Let's let them disable it,
but keep it enabled by default.
This commit is contained in:
Iain Lane
2018-04-24 12:31:44 +01:00
committed by Aleksander Morgado
parent 7e94928301
commit 5072db3418
1 changed files with 7 additions and 4 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
m4/compiler_warnings.m4
View File

@@ -1,11 +1,11 @@
AC_DEFUN([NM_COMPILER_WARNINGS], AC_DEFUN([NM_COMPILER_WARNINGS],
[AC_ARG_ENABLE(more-warnings, [AC_ARG_ENABLE(more-warnings,
AS_HELP_STRING([--enable-more-warnings], [Maximum compiler warnings]), AS_HELP_STRING([--enable-more-warnings], [Possible values: no/yes/error]),
set_more_warnings="$enableval",set_more_warnings=yes) set_more_warnings="$enableval",set_more_warnings=error)
AC_MSG_CHECKING(for more warnings, including -Werror) AC_MSG_CHECKING(for more warnings)
if test "$GCC" = "yes" -a "$set_more_warnings" != "no"; then if test "$GCC" = "yes" -a "$set_more_warnings" != "no"; then
AC_MSG_RESULT(yes) AC_MSG_RESULT(yes)
CFLAGS="-Wall -Werror -std=gnu89 $CFLAGS" CFLAGS="-Wall -std=gnu89 $CFLAGS"
for option in -Wmissing-declarations -Wmissing-prototypes \ for option in -Wmissing-declarations -Wmissing-prototypes \
-Wdeclaration-after-statement -Wstrict-prototypes \ -Wdeclaration-after-statement -Wstrict-prototypes \
@@ -26,6 +26,9 @@ if test "$GCC" = "yes" -a "$set_more_warnings" != "no"; then
unset SAVE_CFLAGS unset SAVE_CFLAGS
done done
unset option unset option
if test "x$set_more_warnings" = xerror; then
CFLAGS="$CFLAGS -Werror"
fi
else else
AC_MSG_RESULT(no) AC_MSG_RESULT(no)
fi fi