From mboxrd@z Thu Jan 1 00:00:00 1970 From: Thomas Monjalon Subject: Re: [PATCH v3 00/13] standard make install Date: Thu, 03 Dec 2015 18:05:55 +0100 Message-ID: <1834388.eHZzREgmng@xps13> References: <1449118929-19962-1-git-send-email-thomas.monjalon@6wind.com> <1449150340-21984-1-git-send-email-thomas.monjalon@6wind.com> <20151203153140.GA11472@bricha3-MOBL3> Mime-Version: 1.0 Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7Bit Cc: dev@dpdk.org To: Bruce Richardson Return-path: Received: from mail-wm0-f44.google.com (mail-wm0-f44.google.com [74.125.82.44]) by dpdk.org (Postfix) with ESMTP id 886D3567E for ; Thu, 3 Dec 2015 18:07:09 +0100 (CET) Received: by wmuu63 with SMTP id u63so30056247wmu.0 for ; Thu, 03 Dec 2015 09:07:09 -0800 (PST) In-Reply-To: <20151203153140.GA11472@bricha3-MOBL3> List-Id: patches and discussions about DPDK List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dev-bounces@dpdk.org Sender: "dev" 2015-12-03 15:31, Bruce Richardson: > On Thu, Dec 03, 2015 at 02:45:27PM +0100, Thomas Monjalon wrote: > > Following the recent discussions, this is a proposal to have a standard > > installation process while keeping compatibility with most of the old > > behaviours. [...] > > Local install example with old (compatible) command: > > > > # make install T=x86_64-native-linuxapp-gcc DESTDIR=install > > > > would be equivalent to: > > > > # make config T=x86_64-native-linuxapp-gcc 0=x86_64-native-linuxapp-gcc > > # make O=x86_64-native-linuxapp-gcc > > # make install O=x86_64-native-linuxapp-gcc prefix= DESTDIR=install > > While I have no huge objections to this patchset, I don't like the fact that > install does completely different things depending upon whether certain variables > are defined or not. I believe the "old" default behaviour for install was > poorly named, and while it's behaviour should be kept, a new name should be given > to it to avoid having the target "install" so heavily overloaded. Yes. It is a first step to fix a common complain. The we can remove the T= syntax or move the "all-in-one feature" to another command name. I suggest "make config-build-install" :)