devtools: expand meson cross compiling test coverage
authorGavin Hu <gavin.hu@arm.com>
Fri, 29 Jun 2018 17:27:38 +0000 (01:27 +0800)
committerThomas Monjalon <thomas@monjalon.net>
Wed, 11 Jul 2018 19:14:21 +0000 (21:14 +0200)
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 <gavin.hu@arm.com>
Reviewed-by: Phil Yang <phil.yang@arm.com>
Reviewed-by: Song Zhu <song.zhu@arm.com>
Acked-by: Bruce Richardson <bruce.richardson@intel.com>
devtools/test-meson-builds.sh

index 662f82d..951c906 100755 (executable)
@@ -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