[Pkg-mc-devel] Bug#489631: mc: bashism in /bin/sh script
Raphael Geissert
atomo64 at gmail.com
Mon Jul 7 02:16:36 UTC 2008
Package: mc
Severity: important
Version: 1:4.6.2~git20080311-2+b1
User: debian-release at 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/mc/extfs/u7z line 48 (should be >word
> 2>&1):
> $P7ZIP d "$1" "$EXFNAME" 2>&1 | grep -q E_NOTIMPL &> /dev/null && \
> possible bashism in ./usr/share/mc/extfs/u7z line 49 (echo -e):
> { echo -e "Function not implemented...\n7z cannot delete from solid
> archive." >&2 ; exit 1 ; }
> possible bashism in ./usr/share/mc/extfs/u7z line 57 (should be >word 2>&1):
> $P7ZIP d "$1" "$EXFNAME"/ 2>&1 | grep -q E_NOTIMPL &> /dev/null && \
> possible bashism in ./usr/share/mc/extfs/u7z line 58 (echo -e):
> { echo -e "Function not implemented...\n7z cannot delete from solid
> archive." >&2 ; exit 1 ; }
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:
http://lists.debian.org/debian-release/2008/01/msg00189.html
For more information please refer to the Debian Policy section 10.4 supporting
this argument at:
http://www.debian.org/doc/debian-policy/ch-files.html#s-scripts
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:
https://wiki.ubuntu.com/DashAsBinSh
[1]http://release.debian.org/lenny/goals.txt
Thank you,
--
Atomo64 - Raphael
Please avoid sending me Word, PowerPoint or Excel attachments.
See http://www.gnu.org/philosophy/no-word-attachments.html
More information about the Pkg-mc-devel
mailing list