[Scummvm-devel] ScummVM report for DACA

[John Willis]

> [...]
> 
> >>> In case you're wondering
> >>> what checkbashims does: It validates Shell scripts for syntax
> >>> elements, which are only provided by bash, but not by a standard POSIX
> shell..
> >>> Such scripts commands should be replaced by portable commands or
> the
> >>> shebang be modified to /bin/bash.
> >>
> >> I am not sure about this, but is "/bin/bash" really safe in this case?
> >> What if your system places the shell in "/usr/bin/bash" would it
> >> still work fine then?
> >
> > True, their might a few such setups.
> >
> >> Anyway it seems only to warn about a potentional bashism, so I guess
> >> one with POSIX shell knowledge should look into it.
> >
> > $RANDOM is bash-specific, see
> > http://wiki.bash-hackers.org/scripting/nonportable
> 
> Yes it is, but it's in a file that is only used by the OpenPandora port,
so this is
> only relevant for that port and its maintainer. So no problem for Debian,
I
> think ;).

It would be me ;).

That said, portability is good and unrequited bash'isms are bad so I'll get
it sorted (well set the correct shebang).
Thanks for the heads up.

John