On Monday 14 January 2013 12:55:29 Dan Douglas wrote:
> On Sunday, January 13, 2013 01:28:42 PM Mike Frysinger wrote:
> > On Sunday 13 January 2013 08:29:43 Michał Górny wrote:
> > > + [[ ${BUILD_DIR} ]] || die "${FUNCNAME}: BUILD_DIR not set."
> > 
> > really should use -n there
> 
> Doesn't matter. Only zsh requires it (and pointlessly disagrees with every
> other implementation).

the point wasn't "will it work".  it's more "how easy is it to glance at code 
and know what it is doing".
-mike

Attachment: signature.asc
Description: This is a digitally signed message part.

Reply via email to