From mboxrd@z Thu Jan 1 00:00:00 1970 From: Gavin Hu Subject: [PATCH v11 6/6] devtools: expand meson cross compiling test coverage Date: Tue, 19 Jun 2018 09:37:46 +0800 Message-ID: <20180619013746.13268-7-gavin.hu@arm.com> References: <20180615102143.12778-1-gavin.hu@arm.com> <20180619013746.13268-1-gavin.hu@arm.com> Cc: nd@arm.com, gavin.hu@arm.com, stable@dpdk.org To: dev@dpdk.org Return-path: In-Reply-To: <20180619013746.13268-1-gavin.hu@arm.com> List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 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, also the build dirs are changed to *-host-$c, indicating the difference of cc used. 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 --- devtools/test-meson-builds.sh | 11 +++++++++-- 1 file changed, 9 insertions(+), 2 deletions(-) diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh index 9bb5b93bd..f0912d109 100755 --- a/devtools/test-meson-builds.sh +++ b/devtools/test-meson-builds.sh @@ -44,11 +44,18 @@ done # test compilation with minimal x86 instruction set build build-x86-default -Dmachine=nehalem +# 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 + # 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 + echo "## ERROR: $c is missing..." + exit 1 fi - build build-$(basename $f | tr '_' '-' | cut -d'-' -f-2) --cross-file $f + export CC="ccache gcc" + build build-$(basename $f | tr '_' '-' | cut -d'-' -f-2)-host-gcc \ + --cross-file $f done -- 2.11.0