Message ID | 20180608163807.66737-7-bruce.richardson@intel.com (mailing list archive) |
---|---|
State | Superseded, archived |
Delegated to: | Thomas Monjalon |
Headers | show |
Series | Enable 32-bit native builds with meson | expand |
Context | Check | Description |
---|---|---|
ci/checkpatch | success | coding style OK |
ci/Intel-compilation | success | Compilation OK |
On 6/8/2018 5:38 PM, Bruce Richardson wrote: > Rather than hard-coding the example app to be built only when a set of > conditions are met, we can simplify things by having the app built when > KNI library itself is available. That saves us duplicating the same set > of restrictions on both library and example app. > > Fixes: 89f0711f9ddf ("examples: build some samples with meson") > > Signed-off-by: Bruce Richardson <bruce.richardson@intel.com> > --- > examples/kni/meson.build | 5 ++--- > 1 file changed, 2 insertions(+), 3 deletions(-) > > diff --git a/examples/kni/meson.build b/examples/kni/meson.build > index 0443ab99b..41b87f8b3 100644 > --- a/examples/kni/meson.build > +++ b/examples/kni/meson.build > @@ -6,9 +6,8 @@ > # To build this example as a standalone application with an already-installed > # DPDK instance, use 'make' > > -if host_machine.system() != 'linux' > - build = false > -endif > +# this app can be built if-and-only-if KNI library is buildable > +build = dpdk_conf.has('LIB_LIBRTE_KNI') s/LIB_LIBRTE_KNI/RTE_LIBRTE_KNI/ Same needs to be updated in drivers/net/kni/meson.build too > deps += ['kni', 'bus_pci'] > sources = files( > 'main.c' >
On Tue, Jul 03, 2018 at 12:47:18AM +0100, Ferruh Yigit wrote: > On 6/8/2018 5:38 PM, Bruce Richardson wrote: > > Rather than hard-coding the example app to be built only when a set of > > conditions are met, we can simplify things by having the app built when > > KNI library itself is available. That saves us duplicating the same set > > of restrictions on both library and example app. > > > > Fixes: 89f0711f9ddf ("examples: build some samples with meson") > > > > Signed-off-by: Bruce Richardson <bruce.richardson@intel.com> > > --- > > examples/kni/meson.build | 5 ++--- > > 1 file changed, 2 insertions(+), 3 deletions(-) > > > > diff --git a/examples/kni/meson.build b/examples/kni/meson.build > > index 0443ab99b..41b87f8b3 100644 > > --- a/examples/kni/meson.build > > +++ b/examples/kni/meson.build > > @@ -6,9 +6,8 @@ > > # To build this example as a standalone application with an already-installed > > # DPDK instance, use 'make' > > > > -if host_machine.system() != 'linux' > > - build = false > > -endif > > +# this app can be built if-and-only-if KNI library is buildable > > +build = dpdk_conf.has('LIB_LIBRTE_KNI') > > s/LIB_LIBRTE_KNI/RTE_LIBRTE_KNI/ > > Same needs to be updated in drivers/net/kni/meson.build too > Thanks, good catch.
diff --git a/examples/kni/meson.build b/examples/kni/meson.build index 0443ab99b..41b87f8b3 100644 --- a/examples/kni/meson.build +++ b/examples/kni/meson.build @@ -6,9 +6,8 @@ # To build this example as a standalone application with an already-installed # DPDK instance, use 'make' -if host_machine.system() != 'linux' - build = false -endif +# this app can be built if-and-only-if KNI library is buildable +build = dpdk_conf.has('LIB_LIBRTE_KNI') deps += ['kni', 'bus_pci'] sources = files( 'main.c'
Rather than hard-coding the example app to be built only when a set of conditions are met, we can simplify things by having the app built when KNI library itself is available. That saves us duplicating the same set of restrictions on both library and example app. Fixes: 89f0711f9ddf ("examples: build some samples with meson") Signed-off-by: Bruce Richardson <bruce.richardson@intel.com> --- examples/kni/meson.build | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-)