[haiku-commits] Re: r38081 - haiku/trunk/data/system/boot

  • From: "Axel Dörfler" <axeld@xxxxxxxxxxxxxxxx>
  • To: haiku-commits@xxxxxxxxxxxxx
  • Date: Fri, 13 Aug 2010 14:35:11 +0200

pulkomandy@xxxxxxxxxxxxxxxxx wrote:
> Log:
> Remove bogus shift : shifting an empty list of arguments is not 
> allowed.
> This allows to boot Haiku with mksh as /bin/sh.

The shift is not bogus, ie. the fix is wrong.

> @@ -10,7 +10,6 @@
>       toLaunch="$1"
>       shift
>       toWaitFor="$1"
> -     shift
>       if [ -f "/boot/$toLaunch" ]
>       then
>               "/boot/$toLaunch" $* &

The remaining arguments are supposed to be forwarded to the launched 
application - this no longer happens here, they are prefixed by the 
contents of $toWaitFor.

Bye,
   Axel.


Other related posts: