[Pkg-sysvinit-devel] Bug#772336: Re: Bug#772336: initscripts: bashism in /bin/sh script

[Raphael Geissert]

Control: severity -1 minor

Hi Steve,

On Saturday 06 December 2014 12:45:19 Steve Langasek wrote:
> Control: affects -1 devscripts
> On Sat, Dec 06, 2014 at 01:56:15PM +0100, Raphael Geissert wrote:
> > checkbashisms' output:
> > > possible bashism in ./etc/init.d/mountnfs.sh line 67 (sleep only takes
> > > one integer):
> > >			  sleep 0.1
[...]
> It's also not something that actually breaks with dash.  /bin/sleep is not
> a required builtin in POSIX shell, and it's not a builtin with *either*
> of bash or dash.
> 
> $ sh -c 'type sleep'
> sleep is /bin/sleep
> $ bash -c 'type sleep'
> sleep is /bin/sleep
> $
> 
> So I think this is a bug in checkbashisms, for complaining about something
> that is in no way a bashism.

It is a utility, which is provided as a built-in by some shells such as 
*ksh. Outside of Debian it is also a command that does not necessarily 
implement anything beyond accepting an integer.

(oh and there's a patch in dash's ML to implement sleep as a built-in 
command).

More info on the topic:
http://rgeissert.blogspot.com/2013/01/a-bashism-week-sleep.html

Cheers,
-- 
Raphael Geissert