Re: PATCH: Generalize spool dir permissions

2008-12-31 Thread Robert Spier
> > spooldir_2.diff: shorter lines, avoid string mode comparison, fix > hard-coded mode in warning Applied in r964 (plus some documentation updates)

Re: PATCH: Generalize spool dir permissions

2008-12-24 Thread Jared Johnson
Robert Spier wrote: Please try and wrap your lines at 80 characters. done... at work we actually have an svn hook to catch long lines and I've come to rely on it... but it looks for lines > 99 chars long :) + $self->log(LOGWARN,"Permissions on spool_dir $Spool_dir are not 2750")

Re: PATCH: Generalize spool dir permissions

2008-12-23 Thread Robert Spier
Jared Johnson wrote: > > Index: plugins/virus/clamdscan > === > --- plugins/virus/clamdscan (revision 961) > +++ plugins/virus/clamdscan (working copy) > @@ -24,8 +24,8 @@ > =item * Enable the "AllowSupplementaryGroups" option

PATCH: Generalize spool dir permissions

2008-12-22 Thread Jared Johnson
Hi, I'm beginning to do a bit of work to try to reduce the forkage between our local qpsmtpd and upstream. We use the clamdscan plugin, which requires 0750 or similar permissions on the spool directory. We modifed Qpsmtpd.pm to display a warning if permissions were something _other_ than 07