From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH v3] eal: remove exec-env directory Date: Tue, 2 Apr 2019 20:57:54 +0100 Message-ID: <6c52aa71-5dcb-1dfe-27f3-57d11dcb5a2f@intel.com> References: <20190401230712.31864-1-thomas@monjalon.net> <20190402193048.27410-1-thomas@monjalon.net> <5ff1fa8a-25d1-7676-1ee7-cf9b4c3973c6@intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: Thomas Monjalon , Bruce Richardson , dev , Anand Rawat To: David Marchand Return-path: Received: from mga09.intel.com (mga09.intel.com [134.134.136.24]) by dpdk.org (Postfix) with ESMTP id 345806C9B for ; Tue, 2 Apr 2019 21:57:58 +0200 (CEST) In-Reply-To: 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 4/2/2019 8:55 PM, David Marchand wrote: > On Tue, Apr 2, 2019 at 9:52 PM Ferruh Yigit wrote: > >> On 4/2/2019 8:46 PM, David Marchand wrote: >>> On Tue, Apr 2, 2019 at 9:30 PM Thomas Monjalon >> wrote: >>> >>>> Only one header file (rte_kni_common.h) was in the sub-directory >>>> include/exec-env/ >>>> This file was installed in a sub-directory of the same name >>>> in the makefile-based build. >>>> Source and install directories are moved as below: >>>> >>>> lib/librte_eal/linux/eal/include/exec-env/ >>>> -> lib/librte_eal/linux/eal/include/ >>>> >>>> build/include/exec-env/ >>>> -> build/include/ >>>> >>>> The consequence is to have a file hierarchy a bit more flat. >>>> >>>> Signed-off-by: Thomas Monjalon >>>> >>> >>> Looks good to me. >>> Reviewed-by: David Marchand >>> Tested-by: David Marchand >>> >>> >>> Bruce, while testing with meson, I noticed this: >>> kernel/linux/kni/meson.build:16: WARNING: Passed invalid keyword argument >>> "console". >>> WARNING: This will become a hard error in the future. >>> WARNING: Unknown keyword arguments in target rte_kni: console >> >> I am not seeing this... >> > > Ok, might be my setup: > [dmarchan@dmarchan dpdk]$ meson --version > 0.47.2 > > What should I check ? I have '0.50.0', but no idea what to check, wait Bruce :) ?