From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751886AbaGMTwu (ORCPT ); Sun, 13 Jul 2014 15:52:50 -0400 Received: from asavdk4.altibox.net ([109.247.116.15]:33132 "EHLO asavdk4.altibox.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750827AbaGMTwn (ORCPT ); Sun, 13 Jul 2014 15:52:43 -0400 Date: Sun, 13 Jul 2014 21:52:36 +0200 From: Sam Ravnborg To: Randy Dunlap Cc: Michal Marek , linux-kbuild , lkml , Thierry Reding , Stephen Rothwell , David Herrmann , Jiri Kosina Subject: Re: [PATCH 2/6] samples: refactor Makefile Message-ID: <20140713195236.GA8711@ravnborg.org> References: <20140713183636.GA17241@ravnborg.org> <1405276974-17323-1-git-send-email-sam@ravnborg.org> <1405276974-17323-2-git-send-email-sam@ravnborg.org> <53C2DD2D.2030902@infradead.org> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <53C2DD2D.2030902@infradead.org> User-Agent: Mutt/1.5.23 (2014-03-12) Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org Hi Randy - thanks for the feedback. On Sun, Jul 13, 2014 at 12:25:33PM -0700, Randy Dunlap wrote: > On 07/13/14 11:42, Sam Ravnborg wrote: > > Use one line per module/program in Makefile. > > This style is easier to read/extend. > > > > Introduce inverse xmas style sorting. > > google search couldn't tell me what that means and I cannot > deduce it from the new Makefile ordering (if there is some ordering). > Please explain. Sort by length - with longest entry first. Like an xmas tree upside-down. For entries with same lenght sort alphabetically. Advocated in some places for includes and other places for local variables. One recent reference is for example this: http://marc.info/?l=linux-netdev&m=140510662905580&w=2 I will rephrase it like this: # Sort entries using inverse xmas style: # Longest entries first, alphabetically when legth is the same. OK? Sam > > Thanks. > > > This may prevent some merge conflicts in the future > > > > Signed-off-by: Sam Ravnborg > > --- > > samples/Makefile | 15 +++++++++++++-- > > 1 file changed, 13 insertions(+), 2 deletions(-) > > > > diff --git a/samples/Makefile b/samples/Makefile > > index 1a60c62..7c08028 100644 > > --- a/samples/Makefile > > +++ b/samples/Makefile > > @@ -1,4 +1,15 @@ > > # Makefile for Linux samples code > > +# Sort entries using inverse xmas style > > +# > > +# kernel modules > > +obj-$(CONFIG_SAMPLES) += hw_breakpoint/ > > +obj-$(CONFIG_SAMPLES) += trace_events/ > > +obj-$(CONFIG_SAMPLES) += kprobes/ > > +obj-$(CONFIG_SAMPLES) += kobject/ > > +obj-$(CONFIG_SAMPLES) += kfifo/ > > +obj-$(CONFIG_SAMPLES) += rpmsg/ > > +obj-$(CONFIG_SAMPLES) += kdb/ > > > > -obj-$(CONFIG_SAMPLES) += kobject/ kprobes/ trace_events/ \ > > - hw_breakpoint/ kfifo/ kdb/ hidraw/ rpmsg/ seccomp/ > > +# user space programs > > +obj-$(CONFIG_SAMPLES) += seccomp/ > > +obj-$(CONFIG_SAMPLES) += hidraw/ > > > > > -- > ~Randy