From mboxrd@z Thu Jan  1 00:00:00 1970
From: Marek Vasut <marek.vasut@gmail.com>
Date: Sat, 3 Dec 2011 10:36:32 +0100
Subject: [U-Boot] [PATCH] MAKEALL: Add -l option to only list build
	targets
In-Reply-To: <201112030419.53711.vapier@gentoo.org>
References: <1322897523-6663-1-git-send-email-marek.vasut@gmail.com>
	<201112030923.28183.marek.vasut@gmail.com>
	<201112030419.53711.vapier@gentoo.org>
Message-ID: <201112031036.32805.marek.vasut@gmail.com>
List-Id: <u-boot.lists.denx.de>
MIME-Version: 1.0
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
To: u-boot@lists.denx.de

> On Saturday 03 December 2011 03:23:28 Marek Vasut wrote:
> > > On Saturday 03 December 2011 02:32:03 Marek Vasut wrote:
> > > > +	if [ "$ONLY_LIST" == 'y' ] ; then return ; fi
> > > 
> > > [ "$ONLY_LIST" = "y" ] && return
> > 
> > I prefer to be explicit.
> 
> my version is "explicit" as well
> 
> the point was more that if statements shouldn't be one-liners like that. 
> if you want to keep it, then unwrap the block.

Is there any such rule?
> -mike

Any other comments ?

M