[dpdk-dev] [dpdk-stable] [PATCH v13 6/6] devtools: expand meson cross compiling test coverage
Gavin Hu
Gavin.Hu at arm.com
Fri Jun 29 19:29:19 CEST 2018
Hi Thomas,
I submitted the v14 patch set to address your two comments!
Any more comments are welcome!
Best Regards,
Gavin
> -----Original Message-----
> From: Thomas Monjalon <thomas at monjalon.net>
> Sent: Thursday, June 28, 2018 5:26 AM
> To: Gavin Hu <Gavin.Hu at arm.com>
> Cc: dev at dpdk.org; nd <nd at arm.com>
> Subject: Re: [dpdk-stable] [PATCH v13 6/6] devtools: expand meson cross
> compiling test coverage
>
> 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
>
>
>
More information about the dev
mailing list