diff options
| author | Ilya Maximets <i.maximets@samsung.com> | 2019-05-28 16:39:46 +0300 |
|---|---|---|
| committer | Ilya Maximets <i.maximets@samsung.com> | 2019-06-06 18:54:57 +0300 |
| commit | dbd4831028ab3ed5bf866a754c3a9ea3725c4d6c (patch) | |
| tree | e60ac14d7c4de8363154fbc34cce38af275ef99a /.travis | |
| parent | 50f48c27e282db4fabdeb737d715d5bbedd1877a (diff) | |
| download | openvswitch-dbd4831028ab3ed5bf866a754c3a9ea3725c4d6c.tar.gz | |
travis: Use enable-Werror knob.
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 <i.maximets@samsung.com>
Acked-by: Ben Pfaff <blp@ovn.org>
Diffstat (limited to '.travis')
| -rwxr-xr-x | .travis/linux-build.sh | 9 |
1 files 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 $*" |