[dpdk-dev,6/7] examples/kni: fix dependency check for building with meson
Checks
Commit Message
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(-)
Comments
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.
@@ -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'