Bug#772410: scilab: bashism in /bin/sh script

[Raphael Geissert]

Package: scilab
Severity: serious
Version: 5.5.1-4
User: debian-release at 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/scinotes line 53 (exit|return with negative
> status code):
>	  exit -1
> possible bashism in ./usr/bin/scinotes line 817 (should be 'b = a'):
>	  if test "$(pwd)" == "/"; then
> possible bashism in ./usr/bin/xcos line 53 (exit|return with negative
> status code):
>	  exit -1
> possible bashism in ./usr/bin/xcos line 817 (should be 'b = a'):
>	  if test "$(pwd)" == "/"; then
> possible bashism in ./usr/bin/scilab line 53 (exit|return with negative
> status code):
>	  exit -1
> possible bashism in ./usr/bin/scilab line 817 (should be 'b = a'):
>	  if test "$(pwd)" == "/"; then
> possible bashism in ./usr/bin/scilab-adv-cli line 53 (exit|return with
> negative status code):
>	  exit -1
> possible bashism in ./usr/bin/scilab-adv-cli line 817 (should be 'b = a'):
>	  if test "$(pwd)" == "/"; 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