Bug#772331: libvisp-dev: bashism in /bin/sh script

[Fabien Spindler]

Hi,

Sorry for the delay.

I have introduced a patch that fixes this bug.
I have also introduced two other patches that improve compat with cmake 3.0.x and X server.
These changes were uploaded to mentors and are available from http://mentors.debian.net/package/visp

Let me know if I have to do something other...
Best regards

PS: All the changes were done on https://github.com/lagadic/visp-debian that is a fork of https://alioth.debian.org/anonscm/git/debian-science/packages/visp.git

Fabien

----- Mail original -----
> De: "Raphael Geissert" <atomo64 at gmail.com>
> À: maintonly at bugs.debian.org
> Envoyé: Samedi 6 Décembre 2014 14:02:29
> Objet: Bug#772331: libvisp-dev: bashism in /bin/sh script
> 
> Package: libvisp-dev
> Severity: minor
> Version: 2.9.0-3+b2
> 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/visp-config line 87 (exit|return with
> > negative status code):
> >     exit -1;
> > possible bashism in ./usr/bin/visp-config line 100 (exit|return with
> > negative status code):
> >	  *) usage; exit -1 ;;
> 
> 
> 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
> 
>