A missing " prevented the DPDK build in the matrix from functioning. Correctly build DPDK as a single, shared library and point the OVS build to the corresponding build directory. Also removes the 'make install' as it is not needed.
Due to incorrect casts in the DPDK headers, we have to disable bad-function-cast warnings as being treated as errors for now. Signed-off-by: Thomas Graf <tg...@noironetworks.com> Co-authored-by: Daniele Di Proietto <ddiproie...@vmware.com> --- .travis.yml | 2 +- .travis/build.sh | 15 +++++++++++---- 2 files changed, 12 insertions(+), 5 deletions(-) diff --git a/.travis.yml b/.travis.yml index fd21e30..b66f821 100644 --- a/.travis.yml +++ b/.travis.yml @@ -8,7 +8,7 @@ before_install: ./.travis/prepare.sh env: - OPTS="--disable-ssl" - TESTSUITE=1 KERNEL=1 OPTS="--with-linux=./linux-3.14.7" - - KERNEL=1 DPDK=1 OPTS="--with-dpdk=./dpdk-1.7.0 + - KERNEL=1 DPDK=1 OPTS="--with-dpdk=./dpdk-1.7.0/build" script: ./.travis/build.sh $OPTS diff --git a/.travis/build.sh b/.travis/build.sh index 56623ea..8680f6e 100755 --- a/.travis/build.sh +++ b/.travis/build.sh @@ -3,6 +3,7 @@ set -o errexit KERNELSRC="" +CFLAGS="-Werror" function install_kernel() { @@ -23,8 +24,9 @@ function install_dpdk() cd dpdk-1.7.0 find ./ -type f | xargs sed -i 's/max-inline-insns-single=100/max-inline-insns-single=400/' make config CC=gcc T=x86_64-native-linuxapp-gcc + sed -ri 's,(RTE_BUILD_SHARED_LIB=).*,\1y,' build/.config + sed -ri 's,(CONFIG_RTE_BUILD_COMBINE_LIBS=).*,\1y,' build/.config make CC=gcc RTE_KERNELDIR=$KERNELSRC - sudo make install CC=gcc T=x86_64-native-linuxapp-gcc RTE_KERNELDIR=$KERNELSRC echo "Installed DPDK source in $(pwd)" cd .. } @@ -38,14 +40,19 @@ if [ "$KERNEL" ] || [ "$DPDK" ]; then install_kernel fi -[ "$DPDK" ] && install_dpdk +[ "$DPDK" ] && { + install_dpdk + # Disregard bad function casts until DPDK is fixed + CFLAGS="$CFLAGS -Wno-error=bad-function-cast" +} configure_ovs $* + if [ $CC = "clang" ]; then - make CFLAGS="-Werror -Wno-error=unused-command-line-argument" + make CFLAGS="$CFLAGS -Wno-error=unused-command-line-argument" else - make CFLAGS="-Werror" C=1 + make CFLAGS="$CFLAGS" C=1 [ "$TESTSUITE" ] && make distcheck fi -- 1.9.3 _______________________________________________ dev mailing list dev@openvswitch.org http://openvswitch.org/mailman/listinfo/dev