From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH 6/7] examples/kni: fix dependency check for building with meson Date: Tue, 3 Jul 2018 00:47:18 +0100 Message-ID: <25eed419-0786-d92d-9aa5-7d8bf7c0c349@intel.com> References: <20180608163807.66737-1-bruce.richardson@intel.com> <20180608163807.66737-7-bruce.richardson@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit To: Bruce Richardson , dev@dpdk.org Return-path: Received: from mga06.intel.com (mga06.intel.com [134.134.136.31]) by dpdk.org (Postfix) with ESMTP id ADACC1B914 for ; Tue, 3 Jul 2018 01:47:21 +0200 (CEST) In-Reply-To: <20180608163807.66737-7-bruce.richardson@intel.com> Content-Language: en-US List-Id: DPDK patches and discussions List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 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 > --- > 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' >