[Scummvm-devel] ScummVM report for DACA

[Max Horn]

Am 04.01.2011 um 21:07 schrieb Moritz Muehlenhoff:

> On 2011-01-04, Johannes Schickel <lordhoto at scummvm.org> wrote:
>> On 01/04/2011 07:49 PM, Moritz Muehlenhoff wrote:

[...]

>>> 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 ;).


Cheers,
Max