> This one time, at band camp, Bruno Tréguier said:
> > Stephen Gran wrote :
> > 
> > >sh -x /etc/init.d/clamav-milter stop
> > 
> > Sure ! Here you are:
> 
> Aha!  If you compare your output:
> 
> > r...@kayak:~# sh -x /etc/init.d/clamav-milter stop
> > + OPTIND=1
> > + log_daemon_msg 'Stopping Sendmail milter plugin for ClamAV' clamav-milter
> > + '[' -z 'Stopping Sendmail milter plugin for ClamAV' ']'
> > + '[' -z clamav-milter ']'
> > + echo -n 'Stopping Sendmail milter plugin for ClamAV: clamav-milter'
> > Stopping Sendmail milter plugin for ClamAV: clamav-milter+ '[' -n -22983 ']'
> > + kill -15 --22983
> 
> With the script in our repo:
> 
>   stop)
>   OPTIND=1
>   log_daemon_msg "Stopping $DESC" "$BASENAME"
>   if [ -n "$PID" ]; then
>     PID=`echo $PID | sed 's/[^0-9]//g'`
>     kill -15 -"$PID" 2>/dev/null || true
> 
> You're missing the line that does the fixup:
>     PID=`echo $PID | sed 's/[^0-9]//g'`
> 
> I wonder how that came to be?  Did you not accept a change to the init
> script on upgrade at some point?  Is it screwed up in the .deb?  Am I
> losing my mind?
> 

I just downloaded a fresh .deb from the Debian servers and its definitely not
screwed up in there. The bug reporter did not provide a Version: pseudo-header,
bug juding from reportbug's info he's using testing aka lenny, and that is
0.94.dfsg.2-1, which I just checked.

We do have the code without the above line in etch(-security), but not in any of
the later versions...

Best,
Michael

Attachment: pgpTGJ4Z36OsD.pgp
Description: PGP signature

Reply via email to