On Mon, Jan 24, 2011 at 05:09:10PM +0200, Teodor MICU wrote:
> Hi,

Hello

> 
> 2011/1/24 Harald Jenny <har...@a-little-linux-box.at>:
> >> 7) You should probably add "-q" for all these executions to avoid
> >> unwanted strings during start/stop/restart.
> >>   "`echo $MILTERSOCKET | grep -v ^inet`"
> >
> > If MILTERSOCKET is checked to contain text too?
> 
> Yes, it does cover the case where MILTERSOCKET contains something but
> not if it is empty. For this I would recommend 'printf' since 'echo
> -n' is not portable and not working with /bin/dash.

I will soon post another version of the patch could you please take a look at
it? As /bin/sh is used for this script dash compatibility is an important
point.

> 
> >> It is unclear to me why you need to change the owner of MILTERSOCKET.
> >
> > Because amavisd-milter running as user can't otherwise remove the socket 
> > when
> > being stopped.
> 
> Well, I was thinking that there should be a better way to create the
> socket with the correct owner and group. This is just an workaround
> for being created by root. I think spamass-millter does a pretty good
> job here since I use it in the default configuration (via a socket
> rather than inet).

Well this workaround is not necessary for sendmail but for postfix.

> 
> Thanks

Thank you
Harald



-- 
To UNSUBSCRIBE, email to debian-bugs-dist-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to