From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from foss.arm.com (foss.arm.com [217.140.101.70]) by dpdk.org (Postfix) with ESMTP id 07CEE1D6AF; Fri, 15 Jun 2018 11:26:22 +0200 (CEST) Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 73E0715BF; Fri, 15 Jun 2018 02:26:21 -0700 (PDT) Received: from net-debian.shanghai.arm.com (net-debian.shanghai.arm.com [10.169.36.53]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPA id B82C73F318; Fri, 15 Jun 2018 02:26:20 -0700 (PDT) From: Gavin Hu To: dev@dpdk.org Cc: nd@arm.com, gavin.hu@arm.com, stable@dpdk.org Date: Fri, 15 Jun 2018 17:26:05 +0800 Message-Id: <20180615092605.12043-7-gavin.hu@arm.com> X-Mailer: git-send-email 2.11.0 In-Reply-To: <20180615092605.12043-1-gavin.hu@arm.com> References: <20180615080120.29253-1-gavin.hu@arm.com> <20180615092605.12043-1-gavin.hu@arm.com> Subject: [dpdk-dev] [PATCH v9 6/6] devtools: expand meson cross compiling test coverage X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Fri, 15 Jun 2018 09:26:22 -0000 The default test script covers only default host cc compiler, either gcc or clang, the fix is to cover both, gcc and clang. And 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 | 13 +++++++++++-- 1 file changed, 11 insertions(+), 2 deletions(-) diff --git a/devtools/test-meson-builds.sh b/devtools/test-meson-builds.sh index 9bb5b93bd..8c33ece93 100755 --- a/devtools/test-meson-builds.sh +++ b/devtools/test-meson-builds.sh @@ -44,11 +44,20 @@ done # test compilation with minimal x86 instruction set build build-x86-default -Dmachine=nehalem +# compile the general v8a also for clang to increase coverage +f=config/arm/arm64_armv8_linuxapp_gcc +export CC="ccache clang" +build build-$(basename $f | tr '_' '-' | cut -d'-' -f-2)-host-clang \ + --cross-file $f + # 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