Re: [dpdk-dev] [PATCH v2] devtools: add test script for meson builds

2018-05-29 Thread Bruce Richardson
On Tue, May 29, 2018 at 12:38:14PM +0200, Thomas Monjalon wrote: > 28/05/2018 15:20, Bruce Richardson: > > On Mon, May 28, 2018 at 12:26:24PM +0200, Thomas Monjalon wrote: > > > 28/05/2018 11:33, Bruce Richardson: > > > > On Sat, May 26, 2018 at 11:32:53AM +0200, Thomas Monjalon wrote: > > > > > 25

Re: [dpdk-dev] [PATCH v2] devtools: add test script for meson builds

2018-05-29 Thread Thomas Monjalon
28/05/2018 15:20, Bruce Richardson: > On Mon, May 28, 2018 at 12:26:24PM +0200, Thomas Monjalon wrote: > > 28/05/2018 11:33, Bruce Richardson: > > > On Sat, May 26, 2018 at 11:32:53AM +0200, Thomas Monjalon wrote: > > > > 25/05/2018 17:18, Bruce Richardson: > > > > > On Fri, May 25, 2018 at 04:51:5

Re: [dpdk-dev] [PATCH v2] devtools: add test script for meson builds

2018-05-28 Thread Bruce Richardson
On Mon, May 28, 2018 at 12:26:24PM +0200, Thomas Monjalon wrote: > 28/05/2018 11:33, Bruce Richardson: > > On Sat, May 26, 2018 at 11:32:53AM +0200, Thomas Monjalon wrote: > > > 25/05/2018 17:18, Bruce Richardson: > > > > On Fri, May 25, 2018 at 04:51:58PM +0200, Thomas Monjalon wrote: > > > > > +c

Re: [dpdk-dev] [PATCH v2] devtools: add test script for meson builds

2018-05-28 Thread Thomas Monjalon
28/05/2018 11:33, Bruce Richardson: > On Sat, May 26, 2018 at 11:32:53AM +0200, Thomas Monjalon wrote: > > 25/05/2018 17:18, Bruce Richardson: > > > On Fri, May 25, 2018 at 04:51:58PM +0200, Thomas Monjalon wrote: > > > > +cd $(dirname $(readlink -m $0))/.. > > > > + > > > I don't think we should f

Re: [dpdk-dev] [PATCH v2] devtools: add test script for meson builds

2018-05-28 Thread Bruce Richardson
On Sat, May 26, 2018 at 11:32:53AM +0200, Thomas Monjalon wrote: > 25/05/2018 17:18, Bruce Richardson: > > On Fri, May 25, 2018 at 04:51:58PM +0200, Thomas Monjalon wrote: > > > +default_path=$PATH > > > + > > > +# Load config options > > > +. $(dirname $(readlink -e $0))/load-devel-config > > > +

Re: [dpdk-dev] [PATCH v2] devtools: add test script for meson builds

2018-05-26 Thread Thomas Monjalon
25/05/2018 17:18, Bruce Richardson: > On Fri, May 25, 2018 at 04:51:58PM +0200, Thomas Monjalon wrote: > > +default_path=$PATH > > + > > +# Load config options > > +. $(dirname $(readlink -e $0))/load-devel-config > > + > > Why is this needed here, it seems to be called before each individual > co

Re: [dpdk-dev] [PATCH v2] devtools: add test script for meson builds

2018-05-25 Thread Bruce Richardson
On Fri, May 25, 2018 at 04:51:58PM +0200, Thomas Monjalon wrote: > From: Bruce Richardson > > To simplify testing with the meson and ninja builds, we can add a script > to set up and do multiple builds. Currently this script sets up: > > * clang and gcc builds > * builds using static and shared

[dpdk-dev] [PATCH v2] devtools: add test script for meson builds

2018-05-25 Thread Thomas Monjalon
From: Bruce Richardson To simplify testing with the meson and ninja builds, we can add a script to set up and do multiple builds. Currently this script sets up: * clang and gcc builds * builds using static and shared linkage for binaries (libs are always built as both) * a build using the low