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:52:12 +0100 Message-ID: <5ff1fa8a-25d1-7676-1ee7-cf9b4c3973c6@intel.com> References: <20190401230712.31864-1-thomas@monjalon.net> <20190402193048.27410-1-thomas@monjalon.net> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: dev , Anand Rawat To: David Marchand , Thomas Monjalon , Bruce Richardson Return-path: Received: from mga12.intel.com (mga12.intel.com [192.55.52.136]) by dpdk.org (Postfix) with ESMTP id 0A84B1B114 for ; Tue, 2 Apr 2019 21:52:15 +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: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...