From 44e830c4e9e8eab8ac96bfe1da7d2ab362cef278 Mon Sep 17 00:00:00 2001 From: Gavin Hu Date: Sat, 30 Jun 2018 01:27:38 +0800 Subject: [PATCH] devtools: expand meson cross compiling test coverage The default test script covers only default host cc compiler, either gcc or clang, the fix is to increase the coverage by adding one more to cover clang and the others for gcc. Fixes: a55277a788 ("devtools: add test script for meson builds") Cc: stable@dpdk.org Signed-off-by: Gavin Hu Reviewed-by: Phil Yang Reviewed-by: Song Zhu Acked-by: Bruce Richardson --- devtools/test-meson-builds.sh | 20 +++++++++++++------- 1 file changed, 13 insertions(+), 7 deletions(-) diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh index 662f82d9bc..951c9067a6 100755 --- a/devtools/test-meson-builds.sh +++ b/devtools/test-meson-builds.sh @@ -45,10 +45,16 @@ done build build-x86-default -Dmachine=nehalem # enable cross compilation if gcc cross-compiler is found -for f in config/arm/arm*gcc ; do - c=aarch64-linux-gnu-gcc - if ! command -v $c >/dev/null 2>&1 ; then - continue - fi - build build-$(basename $f | tr '_' '-' | cut -d'-' -f-2) --cross-file $f -done +c=aarch64-linux-gnu-gcc +if command -v $c >/dev/null 2>&1 ; then + # compile the general v8a also for clang to increase coverage + export CC="ccache clang" + build build-arm64-host-clang --cross-file \ + config/arm/arm64_armv8_linuxapp_gcc + + for f in config/arm/arm*gcc ; do + export CC="ccache gcc" + build build-$(basename $f | tr '_' '-' | cut -d'-' -f-2) \ + --cross-file $f + done +fi -- 2.20.1