Patch to default MailScanner.conf for correct SpamAssassin Lo cal State Dir

[Randal, Phil]

It was required with an earlier 3.1.x release, maybe it isn't now, as
your research suggests.  Theo's patch was committed on August 20th, so
that would seem to relate to SA
version 3.1.5 and later.

Cheers,

Phil

--
Phil Randal
Network Engineer
Herefordshire Council
Hereford, UK  

> -----Original Message-----
> From: mailscanner-bounces at lists.mailscanner.info 
> [mailto:mailscanner-bounces at lists.mailscanner.info] On Behalf 
> Of Stef Morrell
> Sent: 23 November 2006 11:29
> To: MailScanner discussion
> Subject: RE: Patch to default MailScanner.conf for correct 
> SpamAssassin Local State Dir
> 
> Randal, Phil [prandal at herefordshire.gov.uk] wrote on :
> > Look through the  archives, see what I did to test it and
> > replicate it if you don't believe me.
> > 
> > *sighs*
> 
> *sighs also*
> 
> Very well...
> 
> Right... ok... that's interesting.
> 
> I find that on my test rig I hadn't uncommented the 
> "Spamassassin Local
> State Dir" value, so it's set to null, which makes me look suitably
> stupid... except...
> 
> This is the output when Local State is commented (ie null) in
> MailScanner.conf:
> 
> [16278] dbg: config: using "/etc/mail/spamassassin" for site rules pre
> files
> [16278] dbg: config: read file /etc/mail/spamassassin/init.pre
> [16278] dbg: config: read file /etc/mail/spamassassin/v310.pre
> [16278] dbg: config: read file /etc/mail/spamassassin/v312.pre
> [16278] dbg: config: using "/var/lib/spamassassin/3.001007" for sys
> rules pre fi
> les
> [16278] dbg: config: read file
> /var/lib/spamassassin/3.001007/updates_spamassass
> in_org.pre
> [16278] dbg: config: using "/var/lib/spamassassin/3.001007" 
> for default
> rules di
> r
> [16278] dbg: config: read file
> /var/lib/spamassassin/3.001007/updates_spamassass
> in_org.cf
> ...
> 
> Seems to have found the updates fine.
> 
> Here is the output when Local State is configured to
> /var/lib/spamassassin
> 
> [16125] dbg: config: using "/etc/mail/spamassassin" for site rules pre
> files
> [16125] dbg: config: read file /etc/mail/spamassassin/init.pre
> [16125] dbg: config: read file /etc/mail/spamassassin/v310.pre
> [16125] dbg: config: read file /etc/mail/spamassassin/v312.pre
> [16125] dbg: config: using "/var/lib/spamassassin/3.001007" for sys
> rules pre fi
> les
> [16125] dbg: config: read file
> /var/lib/spamassassin/3.001007/updates_spamassass
> in_org.pre
> [16125] dbg: config: using "/var/lib/spamassassin/3.001007" 
> for default
> rules di
> r
> [16125] dbg: config: read file
> /var/lib/spamassassin/3.001007/updates_spamassass
> in_org.cf
> ...
> 
> Identical!!
> 
> And for completeness, here with Local State set to /var/lib
> 
> [16672] dbg: config: using "/etc/mail/spamassassin" for site rules pre
> files
> [16672] dbg: config: read file /etc/mail/spamassassin/init.pre
> [16672] dbg: config: read file /etc/mail/spamassassin/v310.pre
> [16672] dbg: config: read file /etc/mail/spamassassin/v312.pre
> [16672] dbg: config: using "/usr/share/spamassassin" for sys rules pre
> files
> [16672] dbg: config: using "/usr/share/spamassassin" for default rules
> dir
> ...
> 
> Didn't find it.
> 
> So.. my hat is off, Phil - you were correct - there is a mistake and
> /var/lib is clearly wrong.
> 
> I pose a new question: Given it behaved correctly with a null value in
> Spamassassin Local State, is the setting required at all?
> 
> Stef
> Stefan Morrell          | Operations Director
> Tel: 0845 3452820       | Alpha Omega Computers Ltd
> Fax: 0845 3452830       | Incorporating Level 5 Internet
> stef at aoc-uk.com         | stef at l5net.net
> 
> > 
> > Phil
> > --
> > Phil Randal
> > Network Engineer
> > Herefordshire Council
> > Hereford, UK
> > 
> >> -----Original Message-----
> >> From: mailscanner-bounces at lists.mailscanner.info
> >> [mailto:mailscanner-bounces at lists.mailscanner.info] On Behalf Of
> >> Stef Morrell Sent: 23 November 2006 10:03
> >> To: MailScanner discussion
> >> Subject: RE: Patch to default MailScanner.conf for correct
> >> SpamAssassin Local State Dir 
> >> 
> >> Hello,
> >> 
> >> Randal, Phil [prandal at herefordshire.gov.uk] wrote:
> >>> As Theo van Dinter notes in
> >>> http://issues.apache.org/SpamAssassin/show_bug.cgi?id=4952#c11
> >>>  , the spamassassin local_state_dir includes /spamassassin, e.g.
> >>> /var/lib/spamassassin, and not /var/lib.
> >> 
> >> Might want to be a bit careful with this. My spamassassin installs,
> >> which are from Julian's most recent bundle have the following code:
> >> 
> >> # note that the CWD takes priority.  This is required in 
> case a user
> >> # is testing a new version of SpamAssassin on a machine with an
> >> older # version installed.  Unless you can come up with a fix for
> >> this that # allows "make test" to work, don't change this.
> >> @default_rules_path = (
> >>   '__local_state_dir__/__version__',
> >>   '__def_rules_dir__',
> >>   '__prefix__/share/spamassassin',
> >>   '/usr/local/share/spamassassin',
> >>   '/usr/share/spamassassin',
> >> );
> >> 
> >> As you can see, /spamassassin is NOT included in the
> >> _local_state_dir_ line. Accordingly, MailScanner.conf should not be
> >> changed at this time. 
> >> 
> >> It will need to be checked again after the next release of SA, I
> >> would guess. 
> >> 
> >> Regards
> >> 
> >> Stef
> >> Stefan Morrell          | Operations Director
> >> Tel: 0845 3452820       | Alpha Omega Computers Ltd
> >> Fax: 0845 3452830       | Incorporating Level 5 Internet
> >> stef at aoc-uk.com         | stef at l5net.net
> >> --
> >> MailScanner mailing list
> >> mailscanner at lists.mailscanner.info
> >> http://lists.mailscanner.info/mailman/listinfo/mailscanner
> >> 
> >> Before posting, read http://wiki.mailscanner.info/posting
> >> 
> >> Support MailScanner development - buy the book off the website!
> >> 
> > --
> > MailScanner mailing list
> > mailscanner at lists.mailscanner.info
> > http://lists.mailscanner.info/mailman/listinfo/mailscanner
> > 
> > Before posting, read http://wiki.mailscanner.info/posting
> > 
> > Support MailScanner development - buy the book off the website!
> -- 
> MailScanner mailing list
> mailscanner at lists.mailscanner.info
> http://lists.mailscanner.info/mailman/listinfo/mailscanner
> 
> Before posting, read http://wiki.mailscanner.info/posting
> 
> Support MailScanner development - buy the book off the website! 
>