laforge has submitted this change. ( https://gerrit.osmocom.org/c/libgtpnl/+/34827?usp=email )

Change subject: configure: add --enable-werror ......................................................................

configure: add --enable-werror

Change-Id: Ife3716c667d2e9b3fcd94d3cdc044d594e7dab5a --- M configure.ac 1 file changed, 28 insertions(+), 0 deletions(-)

Approvals: pespin: Looks good to me, but someone else must approve laforge: Looks good to me, approved Jenkins Builder: Verified

diff --git a/configure.ac b/configure.ac index e7d9888..a16f40b 100644 --- a/configure.ac +++ b/configure.ac @@ -53,6 +53,25 @@ CPPFLAGS="$CPPFLAGS -fsanitize=address -fsanitize=undefined" fi

+AC_ARG_ENABLE(werror, + [AS_HELP_STRING( + [--enable-werror], + [Turn all compiler warnings into errors, with exceptions: + a) deprecation (allow upstream to mark deprecation without breaking builds); + b) "#warning" pragmas (allow to remind ourselves of errors without breaking builds) + ] + )], + [werror=$enableval], [werror="no"]) +if test x"$werror" = x"yes" +then + WERROR_FLAGS="-Werror" + WERROR_FLAGS+=" -Werror=implicit-int -Werror=int-conversion -Werror=old-style-definition" + WERROR_FLAGS+=" -Wno-error=deprecated -Wno-error=deprecated-declarations" + WERROR_FLAGS+=" -Wno-error=cpp" # "#warning" + CFLAGS="$CFLAGS $WERROR_FLAGS" + CPPFLAGS="$CPPFLAGS $WERROR_FLAGS" +fi + AC_SUBST([CPPFLAGS]) AC_SUBST([CFLAGS]) AC_CONFIG_FILES([Makefile src/Makefile include/Makefile include/libgtpnl/Makefile include/linux/Makefile tools/Makefile libgtpnl.pc])