[pkg-fso-maint] install.sh - please change the shebang to #!/bin/bash

Steffen Möller steffen_moeller at gmx.de
Thu Jan 21 15:41:42 UTC 2010


Jonas Smedegaard wrote:
> On Thu, Jan 21, 2010 at 01:19:49AM +0100, Gilles Filippini wrote:
>> Hi,
>>
>> "read -t" is a bashism which triggers an error when /bin/sh is a link to
>> dash, as in qtmoko v16.
>>
>> Please change the shebang to force using bash.
> 
> Or better (if possible - I haven't read the actual code) patch the
> script to not use bashisms - as that will require less memory and be
> slightly faster, even when using /bin/sh symlinked to bash.

If I had had a spontaneous idea then I would have done that. However, I
did not even do a "man read". install.sh should be merged with the
Debian Installer and not exist by itself, and with most time spent in
cdebootstrap I did not care about optimising the rest. The time saved is
most certainly less than what I would have invested for the test run.

Regards,

Steffen




More information about the pkg-fso-maint mailing list