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

Bug#465376: varkon: bashism in /bin/sh script



Package: varkon
Severity: important
Version: 1.18A-3
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 a /bin/sh script making 
use of a bashism.

checkbashisms' output:
> possible bashism in ./usr/bin/varkon line 23 (echo -e):
>         echo -e "\t$1"
> possible bashism in ./usr/bin/varkon line 183 (echo -e):
>     echo -e "Configuration:\tcurrent  (old)"
> possible bashism in ./usr/bin/varkon line 184 (echo -e):
>     echo -e "`if [ "$NEW_HTML_VIEWER" != "$HTML_VIEWER" ]; then echo -n
> '*'; else echo -n " "; fi`1 html_viewer:\t$NEW_HTML_VIEWER  ($HTML_VIEWER)"
> possible bashism in ./usr/bin/varkon line 185 (echo -e):
>     echo -e "`if [ "$NEW_MBS_EDITOR" != "$MBS_EDITOR" ]; then echo -n '*';
> else echo -n " "; fi`2 mbseditor:\t$NEW_MBS_EDITOR  ($MBS_EDITOR)" possible
> bashism in ./usr/bin/varkon line 186 (echo -e):
>     echo -e "`if [ "$NEW_MBS_EMULATOR" != "$MBS_EMULATOR" ]; then echo -n
> '*'; else echo -n " "; fi`3 mbsemulator:\t$NEW_MBS_EMULATOR 
> ($MBS_EMULATOR)" possible bashism in ./usr/bin/varkon line 339 (should be
> '.', not 'source'): source $SYSINI
> possible bashism in ./usr/bin/varkon line 398 (should be '.', not
> 'source'): source $USERINI


Not using bash as /bin/sh would 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
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



Reply to: