← Back to team overview

checkbox-dev team mailing list archive

Bug#772294: Bug#772294: plainbox-provider-checkbox: bashism in /bin/sh script

 

Thanks for reporting this problem.

I will correct it but it will take a while before the updated package
hits the Debian archive.

Best regards
ZK

On Sat, Dec 6, 2014 at 2:26 PM, Raphael Geissert <atomo64@xxxxxxxxx> wrote:
> Package: plainbox-provider-checkbox
> Severity: important
> Version: 0.4-1
> User: debian-release@xxxxxxxxxxxxxxxx
> 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/lib/2013.com.canonical.certification:checkbox/bin/wireless_ext line 3
>> (should be >word 2>&1):
>> if lsmod | grep "80211" &> /dev/null ;
>> possible bashism in
>> ./usr/lib/2013.com.canonical.certification:checkbox/bin/wireless_ext line 6
>> (should be >word 2>&1):
>> if iwconfig 2>&1 | grep "IEEE 802.11" &> /dev/null ;
>
>
> 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
>
> --
> Mailing list: https://launchpad.net/~checkbox-dev
> Post to     : checkbox-dev@xxxxxxxxxxxxxxxxxxx
> Unsubscribe : https://launchpad.net/~checkbox-dev
> More help   : https://help.launchpad.net/ListHelp


References