From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mx1.redhat.com (mx1.redhat.com [209.132.183.28]) by dpdk.org (Postfix) with ESMTP id 2CE138DB3 for ; Wed, 2 Dec 2015 10:33:59 +0100 (CET) Received: from int-mx10.intmail.prod.int.phx2.redhat.com (int-mx10.intmail.prod.int.phx2.redhat.com [10.5.11.23]) by mx1.redhat.com (Postfix) with ESMTPS id 36F9749DAF; Wed, 2 Dec 2015 09:33:58 +0000 (UTC) Received: from sopuli.koti.laiskiainen.org (vpn1-7-93.ams2.redhat.com [10.36.7.93]) by int-mx10.intmail.prod.int.phx2.redhat.com (8.14.4/8.14.4) with ESMTP id tB29XuRO024335; Wed, 2 Dec 2015 04:33:57 -0500 To: Mario Carrillo , dev@dpdk.org 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> From: Panu Matilainen Message-ID: <565EBB04.60909@redhat.com> Date: Wed, 2 Dec 2015 11:33:56 +0200 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.3.0 MIME-Version: 1.0 In-Reply-To: <1448998753-26599-1-git-send-email-mario.alfredo.c.arevalo@intel.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-Scanned-By: MIMEDefang 2.68 on 10.5.11.23 Cc: jos.c.venegas.munoz@intel.com 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 09:33:59 -0000 On 12/01/2015 09:39 PM, Mario Carrillo wrote: > DPDK package lacks of a mechanism to install libraries, headers > applications, kernel modules and sdk files to a file system tree. > This patch set allows to install files based on the next > proposal: > http://www.freedesktop.org/software/systemd/man/file-hierarchy.html > > v8: > > When "make install" is invoked if "T" variable is defined, > the installation process will have the current > behaviour, else "install-fhs" rule will be called. > > Using rules support is possible to do the next steps: > > make config T= > make > make > > Modify the makefile target to specify the files > that will be installed using a rule: > > * make install-bin (install app files)(dafault path bindir=$(exec_prefix)/bin). > > * make install-headers (install headers)(dafault path includedir=$(prefix)/include/dpdk). > > * make install-lib (install libraries)(dafault path libdir=$(exec_prefix)/lib). > > * make install-doc (install documentation)(dafault path docdir=$(datarootdir)/doc/dpdk). > > * make install-mod (install modules)(dafault path if RTE_EXEC_ENV=linuxapp then > kerneldir=/lib/modules/$(uname -r)/extra/drivers/dpdk else kerneldir=/boot/modules). > > * make install-sdk (install headers, makefiles, scripts,examples and > config files) (default path sdkdir=$(datadir)/share/dpdk). > > * make install-fhs (install libraries, modules, app files, tools and documentation). > > * make install (if T is defined current behaviour, else it will call install-fhs rule). > > The following defaults apply: > > prefix=/usr/local > exec_prefix=$(prefix) > datarootdir=$(prefix)/share > > All path variables can be overridden and all targets can use the "DESTDIR" > variable. > > Furthermore this information is added to documentation. Overall, does what it promises. One point I just realized from comparing with Thomas' variant is that this by default installs documentation sources, ie the raw .rst files and does not include any "compiled" formats even if they exist. It might be better to leave docs out by default as Thomas' version does. One way of achieving that is only install docs if $(RTE_OUTPUT)/doc, and only install anything in that directory. That way you have to request doc generation specifically with "make doc" first (which has quite some build-dependencies so you might not always wnat it), and only the compiled docs get installed. Or something like that. - Panu -