[Pkg-lustre-maintainers] Bug#581121: lustre-tests: bashism in /bin/sh script

Raphael Geissert atomo64 at gmail.com
Tue May 11 05:40:40 UTC 2010


Package: lustre-tests
Severity: normal
Version: 1.8.2-1
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 a /bin/sh script making use
of a bashism.

checkbashisms' output:
> possible bashism in ./usr/lib/lustre/tests/lnet-selftest.sh line 69
> (${parm/?/pat[/str]}):
>     local nc=$(echo ${clients//,/ } | wc -w)
> possible bashism in ./usr/lib/lustre/tests/lnet-selftest.sh line 70
> (${parm/?/pat[/str]}):
>     local ns=$(echo ${servers//,/ } | wc -w)
> possible bashism in ./usr/lib/lustre/tests/lnet-selftest.sh line 145
> ($PIPESTATUS):
>     rc=${PIPESTATUS[0]}
> possible bashism in ./usr/lib/lustre/tests/lnet-selftest.sh line 145 (bash
> arrays, ${name[0|*|@]}):
>     rc=${PIPESTATUS[0]}


Not using bash (or a Debian Policy compliant shell interpreter that doesn't
provide such an extra feature) as /bin/sh is likely to lead to errors or
unexpected behaviours. Please be aware that dash is now the default /bin/sh.

Hints about how to fix bashisms can be found at:
https://wiki.ubuntu.com/DashAsBinSh

If you still don't know how to fix the bashisms don't hesitate to reply to
this email, or tag the bug as 'help'.

Thank you,
Raphael Geissert





More information about the Pkg-lustre-maintainers mailing list