Bug#530047: asterisk: bashism in /bin/sh script

Raphael Geissert atomo64 at gmail.com
Sun May 24 20:16:43 UTC 2009


On Saturday 23 May 2009 10:33:58 Tzafrir Cohen wrote:
> Something new I get in 1.6.1 (and also in upstream trunk)
>
>   possible bashism in - line 92 (ulimit):
>           ulimit -n $MAXFILES
>   possible bashism in - line 103 (ulimit):
>   ulimit -c unlimited
>
> I can't think of a simple workaround. OTOH, dash seems to be fine with
> that

Exactly the reason why I didn't include it in the report. As a matter of fact, 
the omitted lines were:
possible bashism in ./usr/sbin/safe_asterisk line 80 (ulimit):
        ulimit -n $MAXFILES
possible bashism in ./usr/sbin/safe_asterisk line 87 (ulimit):
ulimit -c unlimited
possible bashism in ./etc/init.d/asterisk line 124 (ulimit):
        ulimit -n $MAXFILES

> (posh isn't , busybox ash in Lenny also seems to be fine with it). 
> Worth fixing? How?

When I proposed the release goal for squeeze I also proposed another goal to 
achieve "full" policy compliance regarding shell scripts by avoiding features 
supported even by dash. Others asked why not try to get those other features 
(e.g. ulimit) required by policy. The discussion is, by now, dead but I will 
forward it to debian-policy.

So, you have two options: leave it the way it is, or catch the error produced 
when ulimit is not implemented.

Cheers,
-- 
Raphael Geissert - Debian Maintainer
www.debian.org - get.debian.net





More information about the Pkg-voip-maintainers mailing list