Bug#530046: asterisk-doc: bashism in /bin/sh script

[Raphael Geissert]

On Saturday 23 May 2009 05:16:05 Tzafrir Cohen wrote:
> On Sat, May 23, 2009 at 12:05:52AM -0500, Raphael Geissert wrote:
[...]
> > checkbashisms' output:
> > > possible bashism in
> > > ./usr/share/doc/asterisk-doc/contrib/init.d/rc.mandriva.zaptel line 32
> > > ('function' is useless):
> > > function probe() {
> > > possible bashism in
> > > ./usr/share/doc/asterisk-doc/contrib/init.d/rc.mandriva.zaptel line 44
> > > ('function' is useless):
> > > function unprobe() {
> > > possible bashism in
> > > ./usr/share/doc/asterisk-doc/contrib/init.d/rc.mandriva.zaptel line 56
> > > ('function' is useless):
> > > function reverse_modules() {
> > > possible bashism in
>
> As their name suggests, those scripts are not intended to be used in a
> Debian system. They are included as the whole contrib directory is
> included.

Ok, thanks. Although it is not the pourpose of this bug report, you might want 
to reconsider whether it is worth at all shipping those files in Debian.

>
> This script specifically has also been removed from upstream as it is
> pointless even there. (in trunk, targeted for 1.6.3, which may or may
> not make it into Squeeze).
>
> So if it really bothers anybody, we can just actively remove that
> script. I think that in this case it is not worth the extra bother.

Closing the report whenever 1.6.3 enters the archive should be enough.

Thanks!

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