From mboxrd@z Thu Jan 1 00:00:00 1970 From: Ferruh Yigit Subject: Re: [PATCH] eal: remove exec-env directory Date: Tue, 2 Apr 2019 15:43:59 +0100 Message-ID: <24ae466d-13f0-b618-ac18-a26d4a5959dd@intel.com> References: <20190401230712.31864-1-thomas@monjalon.net> <20190402093933.GD1470@bricha3-MOBL.ger.corp.intel.com> Mime-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit Cc: dev@dpdk.org, david.marchand@redhat.com, anand.rawat@intel.com To: Bruce Richardson , Thomas Monjalon Return-path: Received: from mga11.intel.com (mga11.intel.com [192.55.52.93]) by dpdk.org (Postfix) with ESMTP id 858622C36 for ; Tue, 2 Apr 2019 16:44:02 +0200 (CEST) In-Reply-To: <20190402093933.GD1470@bricha3-MOBL.ger.corp.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 4/2/2019 10:39 AM, Bruce Richardson wrote: > On Tue, Apr 02, 2019 at 01:07:12AM +0200, 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. >> 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 >> --- >> devtools/check-includes.sh | 1 - >> kernel/linux/kni/kni_dev.h | 2 +- >> kernel/linux/kni/kni_fifo.h | 2 +- >> kernel/linux/kni/kni_misc.c | 2 +- >> kernel/linux/kni/kni_net.c | 2 +- >> lib/librte_eal/freebsd/eal/Makefile | 3 +-- >> lib/librte_eal/linux/eal/Makefile | 3 +-- >> .../linux/eal/include/{exec-env => }/rte_kni_common.h | 0 >> lib/librte_kni/rte_kni.c | 2 +- >> lib/librte_kni/rte_kni.h | 2 +- >> 10 files changed, 8 insertions(+), 11 deletions(-) >> rename lib/librte_eal/linux/eal/include/{exec-env => }/rte_kni_common.h (100%) >> > +1 for flattening our EAL hierarchy. +1