From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga01.intel.com (mga01.intel.com [192.55.52.88]) by dpdk.org (Postfix) with ESMTP id E81B88E56 for ; Thu, 3 Dec 2015 00:04:41 +0100 (CET) Received: from orsmga001.jf.intel.com ([10.7.209.18]) by fmsmga101.fm.intel.com with ESMTP; 02 Dec 2015 15:04:41 -0800 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.20,375,1444719600"; d="scan'208";a="833034194" Received: from fmsmsx108.amr.corp.intel.com ([10.18.124.206]) by orsmga001.jf.intel.com with ESMTP; 02 Dec 2015 15:04:40 -0800 Received: from fmsmsx101.amr.corp.intel.com (10.18.124.199) by FMSMSX108.amr.corp.intel.com (10.18.124.206) with Microsoft SMTP Server (TLS) id 14.3.248.2; Wed, 2 Dec 2015 15:04:38 -0800 Received: from fmsmsx107.amr.corp.intel.com ([169.254.6.71]) by fmsmsx101.amr.corp.intel.com ([169.254.1.182]) with mapi id 14.03.0248.002; Wed, 2 Dec 2015 15:04:38 -0800 From: "Arevalo, Mario Alfredo C" To: Panu Matilainen , "dev@dpdk.org" Thread-Topic: [dpdk-dev] [PATCH v8 00/11] Add installation rules for dpdk files. Thread-Index: AQHRLG/sWg3C6ygEcUa7Wyu5K9DIMp639l0AgABcOHM= Date: Wed, 2 Dec 2015 23:04:37 +0000 Message-ID: <6594B51DBE477C48AAE23675314E6C460F784F00@fmsmsx107.amr.corp.intel.com> References: <1442608390-12537-1-git-send-email-mario.alfredo.c.arevalo@intel.com> <1448998753-26599-1-git-send-email-mario.alfredo.c.arevalo@intel.com>, <565EBB04.60909@redhat.com> In-Reply-To: <565EBB04.60909@redhat.com> Accept-Language: en-US Content-Language: en-US X-MS-Has-Attach: X-MS-TNEF-Correlator: x-originating-ip: [10.19.9.46] Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: quoted-printable MIME-Version: 1.0 Cc: "Venegas Munoz, Jos C" Subject: Re: [dpdk-dev] [PATCH v8 00/11] Add installation rules for dpdk files. X-BeenThere: dev@dpdk.org X-Mailman-Version: 2.1.15 Precedence: list List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 02 Dec 2015 23:04:42 -0000 Thank you, I'm going to take note about it for a version number 9 :)=0A= =0A= Thanks.=0A= Mario.=0A= ________________________________________=0A= From: Panu Matilainen [pmatilai@redhat.com]=0A= Sent: Wednesday, December 02, 2015 1:33 AM=0A= To: Arevalo, Mario Alfredo C; dev@dpdk.org=0A= Cc: Venegas Munoz, Jos C=0A= Subject: Re: [dpdk-dev] [PATCH v8 00/11] Add installation rules for dpdk fi= les.=0A= =0A= On 12/01/2015 09:39 PM, Mario Carrillo wrote:=0A= > DPDK package lacks of a mechanism to install libraries, headers=0A= > applications, kernel modules and sdk files to a file system tree.=0A= > This patch set allows to install files based on the next=0A= > proposal:=0A= > http://www.freedesktop.org/software/systemd/man/file-hierarchy.html=0A= >=0A= > v8:=0A= >=0A= > When "make install" is invoked if "T" variable is defined,=0A= > the installation process will have the current=0A= > behaviour, else "install-fhs" rule will be called.=0A= >=0A= > Using rules support is possible to do the next steps:=0A= >=0A= > make config T=3D=0A= > make=0A= > make =0A= >=0A= > Modify the makefile target to specify the files=0A= > that will be installed using a rule:=0A= >=0A= > * make install-bin (install app files)(dafault path bindir=3D$(exec_prefi= x)/bin).=0A= >=0A= > * make install-headers (install headers)(dafault path includedir=3D$(pref= ix)/include/dpdk).=0A= >=0A= > * make install-lib (install libraries)(dafault path libdir=3D$(exec_prefi= x)/lib).=0A= >=0A= > * make install-doc (install documentation)(dafault path docdir=3D$(dataro= otdir)/doc/dpdk).=0A= >=0A= > * make install-mod (install modules)(dafault path if RTE_EXEC_ENV=3Dlinux= app then=0A= > kerneldir=3D/lib/modules/$(uname -r)/extra/drivers/dpdk = else kerneldir=3D/boot/modules).=0A= >=0A= > * make install-sdk (install headers, makefiles, scripts,examples and=0A= > config files) (default path sdkdir=3D$(datadir)/share/dp= dk).=0A= >=0A= > * make install-fhs (install libraries, modules, app files, tools and doc= umentation).=0A= >=0A= > * make install (if T is defined current behaviour, else it will call inst= all-fhs rule).=0A= >=0A= > The following defaults apply:=0A= >=0A= > prefix=3D/usr/local=0A= > exec_prefix=3D$(prefix)=0A= > datarootdir=3D$(prefix)/share=0A= >=0A= > All path variables can be overridden and all targets can use the "DESTDIR= "=0A= > variable.=0A= >=0A= > Furthermore this information is added to documentation.=0A= =0A= Overall, does what it promises.=0A= =0A= One point I just realized from comparing with Thomas' variant is that=0A= this by default installs documentation sources, ie the raw .rst files=0A= and does not include any "compiled" formats even if they exist.=0A= =0A= It might be better to leave docs out by default as Thomas' version does.=0A= One way of achieving that is only install docs if $(RTE_OUTPUT)/doc, and=0A= only install anything in that directory. That way you have to request=0A= doc generation specifically with "make doc" first (which has quite some=0A= build-dependencies so you might not always wnat it), and only the=0A= compiled docs get installed. Or something like that.=0A= =0A= - Panu -=0A=