Re: [Xen-devel] [PATCH] Minor fixes for non-Linux platforms

[Keir Fraser <Keir.Fraser@xxxxxxxxxxxx>]

On 13/10/06 10:56, "Bastian Blank" <bastian@xxxxxxxxxxxx> wrote:

>> diff -r f14a67a35bec tools/check/check_brctl
>> --- a/tools/check/check_brctl Thu Oct 12 17:53:51 2006 +0100
>> +++ b/tools/check/check_brctl Thu Oct 12 10:17:33 2006 +0200
>> @@ -1,4 +1,4 @@
>> -#!/bin/bash
>> +#!/bin/sh
> 
> Thats a bad idea. Some of the scripts uses non-POSIX extensions and
> /bin/sh is normaly only defined as POSIX compilant.

The scripts are tiny. We should stop them using those non-compliant
extensions. We use /bin/sh in just about all our other scripts (and some of
those are *way* more complex).

 -- Keir



_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxx
http://lists.xensource.com/xen-devel