laforge has submitted this change. ( https://gerrit.osmocom.org/c/osmocom-bb/+/30012 )

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

trxcon: configure.ac: add --enable-werror

Change-Id: Id9b561f59599e5eb0267e3baaa33bf9cdde22ea3 --- M src/host/trxcon/configure.ac 1 file changed, 22 insertions(+), 0 deletions(-)

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

diff --git a/src/host/trxcon/configure.ac b/src/host/trxcon/configure.ac index 46991a2..554caab 100644 --- a/src/host/trxcon/configure.ac +++ b/src/host/trxcon/configure.ac @@ -32,6 +32,28 @@ 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_MSG_RESULT([CFLAGS="$CFLAGS"]) +AC_MSG_RESULT([CPPFLAGS="$CPPFLAGS"]) + dnl Checks for typedefs, structures and compiler characteristics

AC_CONFIG_MACRO_DIRS([m4])