Bug#690617: mjpegtools: bashism in /bin/sh script

Raphael Geissert atomo64 at gmail.com
Tue Oct 16 02:10:23 UTC 2012


Package: mjpegtools
Version: 2.0.0+debian-1
Severity: important
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/bin/lav2mpeg line 255 (let ...):
>let MOPTIND=OPTIND-LAVRC_COUNT
>possible bashism in ./usr/bin/lav2mpeg line 545 ($SECONDS):
>STARTALL=$SECONDS
>possible bashism in ./usr/bin/lav2mpeg line 549 ($SECONDS):
>   START=$SECONDS
>possible bashism in ./usr/bin/lav2mpeg line 567 ($SECONDS):
>      diff=$(getTimeDiff $START $SECONDS)
>possible bashism in ./usr/bin/lav2mpeg line 568 ($SECONDS):
>      elapsed=$(expr $SECONDS - $START)
>possible bashism in ./usr/bin/lav2mpeg line 578 ($SECONDS):
>END=$SECONDS
>possible bashism in ./usr/bin/lav2mpeg line 579 ($SECONDS):
>diff=$(getTimeDiff $STARTALL $SECONDS)
>possible bashism in ./usr/bin/lav2mpeg line 580 ($SECONDS):
>temp=$(expr $SECONDS - $STARTALL)

Not using bash (or a Debian Policy conformant shell interpreter which 
doesn't provide such an extra feature) as /bin/sh is likely to lead to 
errors or unexpected behaviours.

You can find hints about how to fix bashisms at:
https://wiki.ubuntu.com/DashAsBinSh

Thank you,
Raphael Geissert



More information about the pkg-multimedia-maintainers mailing list