Unlike manual injecting of "-Werror" to CFLAGS, '--enable-Werror' enables failure on "sparse" and flake8 warnings. At least we wasn't notified about flake8 warnings previously.
Signed-off-by: Ilya Maximets <[email protected]> --- .travis/linux-build.sh | 9 +++------ 1 file changed, 3 insertions(+), 6 deletions(-) diff --git a/.travis/linux-build.sh b/.travis/linux-build.sh index c6d9ae120..79ba4009e 100755 --- a/.travis/linux-build.sh +++ b/.travis/linux-build.sh @@ -4,9 +4,9 @@ set -o errexit set -x KERNELSRC="" -CFLAGS="-Werror" +CFLAGS="" SPARSE_FLAGS="" -EXTRA_OPTS="" +EXTRA_OPTS="--enable-Werror" TARGET="x86_64-native-linuxapp-gcc" function install_kernel() @@ -47,7 +47,7 @@ function install_kernel() KERNELSRC=$(pwd) if [ ! "$DPDK" ] && [ ! "$DPDK_SHARED" ]; then - EXTRA_OPTS="--with-linux=$(pwd)" + EXTRA_OPTS="${EXTRA_OPTS} --with-linux=$(pwd)" fi echo "Installed kernel source in $(pwd)" cd .. @@ -100,9 +100,6 @@ if [ "$DPDK" ] || [ "$DPDK_SHARED" ]; then fi EXTRA_OPTS="$EXTRA_OPTS --with-dpdk=$(pwd)/dpdk-$DPDK_VER/build" fi -if [ "$CC" != "clang" ]; then - SPARSE_FLAGS="$SPARSE_FLAGS -Wsparse-error" -fi OPTS="$EXTRA_OPTS $*" -- 2.17.1 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
