#772243 fvwm-crystal: bashism in /bin/sh script

Package:
fvwm-crystal
Source:
fvwm-crystal
Submitter:
Raphael Geissert
Date:
2014-12-06 13:44:58 UTC
Severity:
minor
#772243#5
Date:
2014-12-06 12:06:53 UTC
From:
To:
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:
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
#772243#10
Date:
2014-12-06 12:26:22 UTC
From:
To:
 ❦  6 décembre 2014 13:06 +0100, Raphael Geissert <atomo64@gmail.com> :

Humm. Both /bin/sleep and sleep in dash seems to work without a problem
with non-integer sleep times.

$ dash -c 'time /bin/sleep 0.5'
0.00user 0.00system 0:00.50elapsed 0%CPU (0avgtext+0avgdata 1776maxresident)k
0inputs+0outputs (0major+87minor)pagefaults 0swaps
$ dash -c 'time sleep 0.5'
0.00user 0.00system 0:00.50elapsed 0%CPU (0avgtext+0avgdata 1840maxresident)k
0inputs+0outputs (0major+92minor)pagefaults 0swaps
#772243#17
Date:
2014-12-06 13:36:50 UTC
From:
To:
Hi,

It's a bug nevertheless. Cf. SUSv3.

Cheers,
#772243#22
Date:
2014-12-06 13:43:35 UTC
From:
To:
 ❦  6 décembre 2014 14:36 +0100, Raphael Geissert <atomo64@gmail.com> :

Sorry, I won't add a patch just to fix a bug that doesn't exist. It even
works with posh since it doesn't have sleep as a builtin.