[pkg-fso-maint] install.sh - please change the shebang to #!/bin/bash
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:
>> "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.
More information about the pkg-fso-maint