[Xenomai-core] [Fwd: Bug#471871: xenomai-runtime: bashism in /bin/sh script]

[Jan Kiszka <jan.kiszka@domain.hid>]

[-- Attachment #1.1: Type: text/plain, Size: 71 bytes --]

Maybe someone with idle cycles left could have a look at this.

Jan

[-- Attachment #1.2: Bug#471871: xenomai-runtime: bashism in /bin/sh script.eml --]
[-- Type: message/rfc822, Size: 6009 bytes --]

From: Raphael Geissert <atomo64@domain.hid>
To: maintonly@domain.hid
Subject: Bug#471871: xenomai-runtime: bashism in /bin/sh script
Date: Thu, 20 Mar 2008 13:39:38 -0600
Message-ID: <200803201339.41302.atomo64@domain.hid>

Package: xenomai-runtime
Severity: important
Version: 2.4.2-2
User: debian-release@domain.hid
Usertags: goal-dash

Hello maintainer,

While performing an archive wide checkbashisms (from the 'devscripts' package) 
check I've found your package containing several /bin/sh scripts making 
use of a bashism.

checkbashisms' output:
> possible bashism in ./usr/bin/xeno-load line 130 (should be >word 2>&1):
>        $sudo kill -SIGINT $children_pids >& /dev/null
> possible bashism in ./usr/bin/xeno-info line 77 (brace expansion):
> ls -l /usr/lib/lib{g,stdc}++.so  2>/dev/null | awk -F. \
> possible bashism in ./usr/bin/xeno-test line 63 (local foo=bar):
>   local neededApplets="awk basename cut date dd dirname egrep grep head
> possible bashism in ./usr/bin/xeno-test line 65 (local foo=bar):
>   local foundAll=1
> possible bashism in ./usr/bin/xeno-test line 81 (local foo=bar):
>   local foundAll=1
> possible bashism in ./usr/bin/xeno-test line 114 (local foo=bar):
>     local jobsct=$1;
> possible bashism in ./usr/bin/xeno-test line 123 (cnt=$((cnt + 1)) does not
> work in dash): jobsct=$((jobsct-1));
> possible bashism in ./usr/bin/xeno-test line 162 (local foo=bar):
>     local filter=" grep -E '$whatconf'"
> possible bashism in ./usr/bin/xeno-test line 277 (local foo=bar):
>     local file=$1
> possible bashism in ./usr/bin/xeno-test line 382 (should be 'b = a'):
>     if test "$res" == "0" ; then

Not using bash 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
https://wiki.ubuntu.com/DashAsBinSh

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

HINT: sometimes these bugs are already fixed in Ubuntu, look at the PTS

[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




[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 254 bytes --]