Subject: Re: CVS commit: pkgsrc/wm/windowmaker/patches
To: None <jlam@netbsd.org>
From: John Darrow <John.P.Darrow@wheaton.edu>
List: pkgsrc-changes
Date: 07/16/2001 15:45:40
Johnny C. Lam <jlam@netbsd.org> wrote:
>	pkgsrc/wm/windowmaker/patches: patch-af
>Fix several nonportable shell constructs:
>	(1) [ doesn't understand ==; use = instead.
>	(2) (( ... )) isn't valid /bin/sh; use [ ... ] instead.
>	(3) ${foo:0:4} to get the first 4 letters of ${foo} isn't valid
>	    /bin/sh, so use a loop to remove letters off the end until
>	    only 4 letters remain.

All these (and ${!tmp} for variable indirection) are bashisms.  Quite
useful, but quite non-portable.

Someone obviously created these scripts on a linux box with bash
masquerading as /bin/sh...

jdarrow

-- 
John Darrow - Senior Technical Specialist               Office: 630/752-5201
Computing Services, Wheaton College, Wheaton, IL 60187  Fax:    630/752-5968
Pager via email: 6303160707@alphapage.airtouch.com      Pager:  630/316-0707
Email:     John.P.Darrow@wheaton.edu