[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#486911: galrey: bashism in /bin/sh script

Package: galrey
Severity: important
Version: 1.0.2-2
User: debian-release@lists.debian.org
Usertags: goal-dash

Hello maintainer,

While performing an archive wide checkbashisms (from the 'devscripts' package) 
check I've found your package containing one or more /bin/sh scripts making 
use of bashisms.

checkbashisms' output:
> possible bashism in ./usr/share/galrey/massscale.sh line 9 (declare):
> declare -a iinfo
> possible bashism in ./usr/share/galrey/massscale.sh line 10 (declare):
> declare -a reqXY
> possible bashism in ./usr/share/galrey/massscale.sh line 14 (bash arrays,
> ${name[0|*|@]}): 
> reqX=${reqXY[0]} 
> possible bashism in ./usr/share/galrey/massscale.sh line 15 (bash arrays,
> ${name[0|*|@]}): 
> reqY=${reqXY[1]} 
> possible bashism in ./usr/share/galrey/massscale.sh line 21 (bash arrays,
> ${name[0|*|@]}):
> realX=${iinfo[0]} 
> possible bashism in ./usr/share/galrey/massscale.sh line 22 (bash arrays,
> ${name[0|*|@]}):
> realY=${iinfo[1]} 
> possible bashism in ./usr/share/galrey/rennum.sh line 8 (should be read
> [-r] variable): 
> read 

Not using bash (or a Debian Policy conformant shell interpreter which does 
provide such an extra feature) as /bin/sh is likely to lead to errors or 
unexpected behaviours.
Please be aware that although bash is currently the default /bin/sh there's a 
release goal for Lenny to make dash the default /bin/sh[1].

If you want more information about dash as /bin/sh, you can read:

For more information please refer to the Debian Policy section 10.4 supporting 
this argument at:

Hints about how to fix bashisms:
Sometimes these bugs are already fixed in Ubuntu, look at the PTS.
If not already fixed you can read:


Thank you,
Atomo64 - Raphael

Please avoid sending me Word, PowerPoint or Excel attachments.
See http://www.gnu.org/philosophy/no-word-attachments.html

Reply to: