Package: tau
Severity: serious
Version: 2.17.3.1.dfsg-3
User: debian-rele...@lists.debian.org
Usertags: goal-dash

Hi,

I've ran checkbashisms (from the 'devscripts' package) over the whole
archive and I found that your package has a /bin/sh script that uses a
"bashism".

checkbashisms' output:
> possible bashism in ./usr/bin/tau_cc line 33 (echo -e):
>     echo -e "  -optVerbose\t\t\tTurn on verbose debugging message"
> possible bashism in ./usr/bin/tau_cc line 34 (echo -e):
>     echo -e "  -optDetectMemoryLeaks\t\tTrack mallocs/frees using TAU's
> memory wrapper"
> possible bashism in ./usr/bin/tau_cc line 35 (echo -e):
>     echo -e "  -optPdtGnuFortranParser\tSpecify the GNU gfortran PDT 
parser
> gfparse instead of f95parse"
> possible bashism in ./usr/bin/tau_cc line 36 (echo -e):
>     echo -e "  -optPdtCleanscapeParser\tSpecify the Cleanscape Fortran
> parser"
> possible bashism in ./usr/bin/tau_cc line 37 (echo -e):
>     echo -e "  -optTauSelectFile=\"\"\t\tSpecify selective instrumentation
> file for tau_instrumentor"
> possible bashism in ./usr/bin/tau_cc line 38 (echo -e):
>     echo -e "  -optPreProcess\t\tPreprocess the source code before 
parsing.
> Uses /usr/bin/cpp -P by default."
> possible bashism in ./usr/bin/tau_cc line 39 (echo -e):
>     echo -e "  -optKeepFiles\t\t\tDoes not remove intermediate .pdb and
> .inst.* files" 
> possible bashism in ./usr/bin/tau_cc line 40 (echo -e):
>     echo -e "  -optShared\t\t\tUse shared library version of TAU."
> possible bashism in ./usr/bin/tau_cc line 41 (echo -e):
>     echo -e "  -optCompInst\t\t\tUse compiler-based instrumentation."
> possible bashism in ./usr/bin/tau_cc line 42 (echo -e):
>     echo -e "  -optPDTInst\t\t\tUse PDT-based instrumentation."
> possible bashism in ./usr/bin/tau_cxx line 33 (echo -e):
>     echo -e "  -optVerbose\t\t\tTurn on verbose debugging message"
> possible bashism in ./usr/bin/tau_cxx line 34 (echo -e):
>     echo -e "  -optDetectMemoryLeaks\t\tTrack mallocs/frees using TAU's
> memory wrapper"
> possible bashism in ./usr/bin/tau_cxx line 35 (echo -e):
>     echo -e "  -optPdtGnuFortranParser\tSpecify the GNU gfortran PDT 
parser
> gfparse instead of f95parse"
> possible bashism in ./usr/bin/tau_cxx line 36 (echo -e):
>     echo -e "  -optPdtCleanscapeParser\tSpecify the Cleanscape Fortran
> parser"
> possible bashism in ./usr/bin/tau_cxx line 37 (echo -e):
>     echo -e "  -optTauSelectFile=\"\"\t\tSpecify selective instrumentation
> file for tau_instrumentor"
> possible bashism in ./usr/bin/tau_cxx line 38 (echo -e):
>     echo -e "  -optPreProcess\t\tPreprocess the source code before 
parsing.
> Uses /usr/bin/cpp -P by default."
> possible bashism in ./usr/bin/tau_cxx line 39 (echo -e):
>     echo -e "  -optKeepFiles\t\t\tDoes not remove intermediate .pdb and
> .inst.* files" 
> possible bashism in ./usr/bin/tau_cxx line 40 (echo -e):
>     echo -e "  -optShared\t\t\tUse shared library version of TAU."
> possible bashism in ./usr/bin/tau_cxx line 41 (echo -e):
>     echo -e "  -optCompInst\t\t\tUse compiler-based instrumentation."
> possible bashism in ./usr/bin/tau_cxx line 42 (echo -e):
>     echo -e "  -optPDTInst\t\t\tUse PDT-based instrumentation."
> possible bashism in ./usr/bin/tau_f90 line 33 (echo -e):
>     echo -e "  -optVerbose\t\t\tTurn on verbose debugging message"
> possible bashism in ./usr/bin/tau_f90 line 34 (echo -e):
>     echo -e "  -optDetectMemoryLeaks\t\tTrack mallocs/frees using TAU's
> memory wrapper"
> possible bashism in ./usr/bin/tau_f90 line 35 (echo -e):
>     echo -e "  -optPdtGnuFortranParser\tSpecify the GNU gfortran PDT 
parser
> gfparse instead of f95parse"
> possible bashism in ./usr/bin/tau_f90 line 36 (echo -e):
>     echo -e "  -optPdtCleanscapeParser\tSpecify the Cleanscape Fortran
> parser"
> possible bashism in ./usr/bin/tau_f90 line 37 (echo -e):
>     echo -e "  -optTauSelectFile=\"\"\t\tSpecify selective instrumentation
> file for tau_instrumentor"
> possible bashism in ./usr/bin/tau_f90 line 38 (echo -e):
>     echo -e "  -optPreProcess\t\tPreprocess the source code before 
parsing.
> Uses /usr/bin/cpp -P by default."
> possible bashism in ./usr/bin/tau_f90 line 39 (echo -e):
>     echo -e "  -optKeepFiles\t\t\tDoes not remove intermediate .pdb and
> .inst.* files" 
> possible bashism in ./usr/bin/tau_f90 line 40 (echo -e):
>     echo -e "  -optShared\t\t\tUse shared library version of TAU."
> possible bashism in ./usr/bin/tau_f90 line 41 (echo -e):
>     echo -e "  -optCompInst\t\t\tUse compiler-based instrumentation."
> possible bashism in ./usr/bin/tau_f90 line 42 (echo -e):
>     echo -e "  -optPDTInst\t\t\tUse PDT-based instrumentation."
> possible bashism in ./usr/bin/tauex line 80 (${foo:3[:1]}):
>               if [ "$i" != "GET_TIME_OF_DAY" -a "${i:0:5}" != "PAPI_" -a
> "${i:0:2}" != "P_" ]; then
> possible bashism in ./usr/bin/tauex line 281 (${foo:3[:1]}):
>     if [ "$c" != "GET_TIME_OF_DAY" -a "${c:0:2}" != "P_" ]; then


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 the default /bin/sh.

Please closely examine the above output and the script, and determine
what the proper severity of the bug is, and adjust it accordingly. If
it's important or greater please hurry to get this fixed for jessie.

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

Thanks in advance,
Raphael Geissert


-- 
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to