From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from mga14.intel.com (mga14.intel.com [192.55.52.115]) by dpdk.org (Postfix) with ESMTP id 3E9608E67 for ; Fri, 2 Oct 2015 13:25:59 +0200 (CEST) Received: from fmsmga001.fm.intel.com ([10.253.24.23]) by fmsmga103.fm.intel.com with ESMTP; 02 Oct 2015 04:25:58 -0700 X-ExtLoop1: 1 X-IronPort-AV: E=Sophos;i="5.17,623,1437462000"; d="scan'208";a="802111477" Received: from bricha3-mobl3.ger.corp.intel.com ([10.237.208.60]) by fmsmga001.fm.intel.com with SMTP; 02 Oct 2015 04:25:47 -0700 Received: by (sSMTP sendmail emulation); Fri, 02 Oct 2015 12:25:46 +0025 Date: Fri, 2 Oct 2015 12:25:46 +0100 From: Bruce Richardson To: Panu Matilainen Message-ID: <20151002112546.GA11604@bricha3-MOBL3> References: <1442608390-12537-1-git-send-email-mario.alfredo.c.arevalo@intel.com> <1443658313-11045-1-git-send-email-mario.alfredo.c.arevalo@intel.com> <1443658313-11045-9-git-send-email-mario.alfredo.c.arevalo@intel.com> <560E6751.6090901@redhat.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <560E6751.6090901@redhat.com> Organization: Intel Shannon Ltd. User-Agent: Mutt/1.5.23 (2014-03-12) Cc: dev@dpdk.org Subject: Re: [dpdk-dev] [PATCH v3 8/8] mk: Add rule for installing runtime 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: Fri, 02 Oct 2015 11:25:59 -0000 On Fri, Oct 02, 2015 at 02:15:29PM +0300, Panu Matilainen wrote: > On 10/01/2015 03:11 AM, Mario Carrillo wrote: > >Add hierarchy-file support to the DPDK libraries, modules, > >binary files, nic bind files and documentation, > >when invoking "make install-fhs" (filesystem hierarchy standard) > >runtime files will be by default installed in: > >$(DESTDIR)/$(BIN_DIR) where BIN_DIR=/usr/bin (binary files) > >$(DESTDIR)/$(SBIN_DIR) where SBIN_DIR=/usr/sbin/dpdk_nic_bind (nic bind > >files) > >$(DESTDIR)/$(DOC_DIR) where DOC_DIR=/usr/share/doc/dpdk (documentation) > >$(DESTDIR)/$(LIB_DIR) (libraries) > >if the architecture is 64 bits then LIB_DIR=/usr/lib64 > >else LIB_DIR=/usr/lib > >$(DESTDIR)/$(KERNEL_DIR) (modules) > >if RTE_EXEC_ENV=linuxapp then > >KERNEL_DIR=/lib/modules/$(uname -r)/build > >else KERNEL_DIR=/boot/modules > >All directory variables mentioned above can be overridden. > >This hierarchy is based on: > >http://www.freedesktop.org/software/systemd/man/file-hierarchy.html > > > > Hmm, I think there's a slight misunderstanding here. > > What I meant earlier by install-sdk and install-fhs is to preserve the > current behavior of "make install" as "make install-sdk" and have "make > install-fhs" behave like normal OSS app on "make install", which installs > everything (both devel and runtime parts) > > This patch series eliminates the current behavior of "make install" > entirely. I personally would not miss it at all, but there likely are people > relying on it since its quite visibly documented and all. So I think the > idea was to introduce a separate FHS-installation target and then deal with > the notion of default behaviors etc separately. > > I guess it was already this way in v2 of the series, apologies for missing > it there. > > - Panu - I also think that having some way to get the old behaviour for those relying on it would be good. Even though it's not ABI affecting, for those compiling from source it would be nice to follow some sort of gradual deprecation process rather than just changing everything in one go. /Bruce