There is no reason anyone should be submitting a diff that has any warnings. Switch over to use -Werror for all compiles.
This commit modifies -Werror as well as fixes a compile warning that has snuck in. Signed-off-by: Donald Sharp <[email protected]> --- configure.ac | 4 ++-- zebra/rt_netlink.c | 1 + 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/configure.ac b/configure.ac index 390a2ce..1bd8c9e 100755 --- a/configure.ac +++ b/configure.ac @@ -175,7 +175,7 @@ else AC_MSG_RESULT([CFLAGS supplied by user]) fi -if test x"${enable_werror}" = x"yes" ; then +if test x"${enable_werror}" != x"no" ; then WERROR="-Werror" fi AC_SUBST(WERROR) @@ -300,7 +300,7 @@ AC_ARG_ENABLE(pcreposix, AC_ARG_ENABLE(fpm, AS_HELP_STRING([--enable-fpm], [enable Forwarding Plane Manager support])) AC_ARG_ENABLE(werror, - AS_HELP_STRING([--enable-werror], [enable -Werror (recommended for developers only)])) + AS_HELP_STRING([--disable-werror], [disable -Werror (recommended for breaking other peoples builds only)])) if test x"${enable_gcc_rdynamic}" != x"no" ; then if test x"${enable_gcc_rdynamic}" = x"yes" -o x"$COMPILER" = x"GCC"; then diff --git a/zebra/rt_netlink.c b/zebra/rt_netlink.c index 0f0f3fe..33fbc08 100644 --- a/zebra/rt_netlink.c +++ b/zebra/rt_netlink.c @@ -1571,6 +1571,7 @@ _netlink_route_debug( routedesc, lookup (nlmsg_str, cmd), prefix2str (p, buf, sizeof(buf)), + zvrf->vrf_id, nexthop_type_to_str (nexthop->type)); } } -- 1.7.10.4 _______________________________________________ Quagga-dev mailing list [email protected] https://lists.quagga.net/mailman/listinfo/quagga-dev
