Re: [dev] [sbase][patch] add [ alias to test, makefile cleanup

From: Evan Gates <>
Date: Fri, 20 Feb 2015 16:11:48 -0800

On Fri, Feb 20, 2015 at 3:36 PM, Nick <> wrote:
> Quoth Evan Gates:
>> sbase-use_pe.diff : use parameter expansion instead of basename when
>> building sbase-box. simpler, shorter, no extra processes
> I don't like this. The sbase Makefile is a really standard, basic,
> make at the moment (almost completely within the OpenGroup makefile
> spec), and AFAIK builds basically everywhere as a result. Parameter
> expansion is something I think of as a bashism, and I doubt is
> available everywhere. To my eyes it's uglier too. Spawning an extra
> process to call basename is less important than readability, though
> of course to an extent that's in the eye of the beholder.

This is POSIX

I think (although I could be wrong) that it's safe to assume we'll be
using a POSIX compliant sh (for loops, redirections, etc.). As for
pretty vs ugly, I prefer the parameter expansion but that's purely
opinion and based on the amount of time I spend in the shell and
writing shell scripts.

Received on Sat Feb 21 2015 - 01:11:48 CET

This archive was generated by hypermail 2.3.0 : Sat Feb 21 2015 - 01:24:07 CET