From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [dpdk-stable] [PATCH v13 6/6] devtools: expand meson cross compiling test coverage Date: Wed, 27 Jun 2018 23:25:49 +0200 Message-ID: <5050805.QkCkagIahT@xps> References: <20180619103657.19186-1-gavin.hu@arm.com> <20180625024913.17219-1-gavin.hu@arm.com> <20180625024913.17219-7-gavin.hu@arm.com> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org, nd@arm.com To: Gavin Hu Return-path: Received: from out1-smtp.messagingengine.com (out1-smtp.messagingengine.com [66.111.4.25]) by dpdk.org (Postfix) with ESMTP id 5E1441C3BD for ; Wed, 27 Jun 2018 23:25:52 +0200 (CEST) In-Reply-To: <20180625024913.17219-7-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" Hi, 25/06/2018 04:49, Gavin Hu: > --- a/devtools/test-meson-builds.sh > +++ b/devtools/test-meson-builds.sh > # enable cross compilation if gcc cross-compiler is found > +c=aarch64-linux-gnu-gcc > +if ! command -v $c >/dev/null 2>&1 ; then > + echo "## ERROR: $c is missing, cross compiling is aborting..." Not having a cross compiler should not be an error. > + return Please, no return, because we could add more compilation tests below, including other cross-compilations. > +fi > + > +# 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 > - 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 > + export CC="ccache gcc" > + build build-$(basename $f | tr '_' '-' | cut -d'-' -f-2) \ > + --cross-file $f > done