Re: "portable" sh is not...

["Christoph Egger" <Christoph.Egger@amd.com>]

On Tuesday 23 October 2007 14:20:14 John Levon wrote:
> On Tue, Oct 23, 2007 at 02:14:38PM +0200, Christoph Egger wrote:
> > > http://xenbits.xensource.com/staging/xen-unstable.hg?rev/08295dc13166
> > >
> > > broke our build. The script is not portable as claimed (e.g.
> > > traditional Bourne shell has no $( ... ) construct).
> >
> > The only one bash-specific thing I could find was the [:alpha:] notion.
> > This changeset makes it work on BSD's /bin/sh.
> > Looks like BSD's /bin/sh is not that traditional... :)

Looking through get-fields.sh, it seems the $( ... ) constructs can be
replaced with backticks. This should make Solaris' /bin/sh happy.

> > > Christoph, why is this necessary? Surely *BSD have /bin/bash
> >
> > No, they have not. They have /bin/sh, /bin/ksh and /bin/csh.
> > bash is a third-party package.
>
> Can we make it use $SHELL? We have that set to bash.

This is an alternative solution to mine above.

Christoph


-- 
AMD Saxony, Dresden, Germany
Operating System Research Center

Legal Information:
AMD Saxony Limited Liability Company & Co. KG
Sitz (Geschäftsanschrift):
   Wilschdorfer Landstr. 101, 01109 Dresden, Deutschland
Registergericht Dresden: HRA 4896
vertretungsberechtigter Komplementär:
   AMD Saxony LLC (Sitz Wilmington, Delaware, USA)
Geschäftsführer der AMD Saxony LLC:
   Dr. Hans-R. Deppe, Thomas McCoy