SA rules_du_jour lint bug: FOUND
MailScanner at ecs.soton.ac.uk
Fri Aug 4 08:44:57 IST 2006
-----BEGIN PGP SIGNED MESSAGE-----
On 3 Aug 2006, at 18:54, Jeff A. Earickson wrote:
> Doh!! Thanks for pointing this out, I had always thought that
> bayes_path was a directory name, not some strange combo of
> directory + file prepend. I fixed my spam.assassin.prefs.conf,
> reverted to the original SA Conf.pm, everything works with
> Rules_du_jour again.
> Could you please modify the spam.assassin.prefs.conf to include
> the following useful comments (diff -c format)?
> *** 83,89 ****
> --- 83,96 ----
> # FSL Note: we need to coordinate the Bayes File Placement
> # With MailWatch
> + # bayes_path should NOT be directory!
> + # The Rules_du_jour script will choke if it is a directory.
> + # It needs to be a full pathname, PLUS a partial filename.
> + # In this example, the trailing "bayes" will be the "bayes*" + #
> files in the directory "/etc/MailScanner/bayes/"
> + # Thanks to Matt Kettler for pointing this out.
> #bayes_path /etc/MailScanner/bayes/bayes
> # This is actually used as a mask, not a raw chmod setting.
> # Thanks for Matt Kettler for spotting this one.
> # Commented out: this if for MailWatch and Exim/Postfix users only.
> Jeff Earickson
> Colby College
> On Thu, 3 Aug 2006, Matt Kettler wrote:
>> Date: Thu, 03 Aug 2006 12:57:10 -0400
>> From: Matt Kettler <mkettler at evi-inc.com>
>> Reply-To: MailScanner discussion <mailscanner at lists.mailscanner.info>
>> To: MailScanner discussion <mailscanner at lists.mailscanner.info>
>> Subject: Re: SA rules_du_jour lint bug: FOUND
>> Jeff A. Earickson wrote:
>>> I upgraded to SA 3.1.4 last night, and continued to be plagued by
>>> lint failures in the rules_du_jour script, like so:
>>> /opt/perl5/bin/spamassassin -p
>>> /opt/MailScanner/etc/spam.assassin.prefs.conf --lint
>>>  warn: config: SpamAssassin failed to parse line,
>>> "/var/spool/spamassassin" is not valid for "bayes_path", skipping:
>>> bayes_path /var/spool/spamassassin
>>> I had this problem in earlier versions of SA, but had worked around
>>> I know this isn't quite right, since the test should be "if NOT a
>>> directory, return INVALID_VALUE". But the code:
>> NO! It should be IF a directory, return invalid. It is NOT valid
>> to specify a
>> directory as a bayes path, because bayes_path is not just a path.
>> It's a path
>> plus partial filename!
>> MailScanner mailing list
>> mailscanner at lists.mailscanner.info
>> 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
> Before posting, read http://wiki.mailscanner.info/posting
> Support MailScanner development - buy the book off the website!
MailScanner at ecs.soton.ac.uk
-----BEGIN PGP SIGNATURE-----
Version: PGP SDK 3.7.0
-----END PGP SIGNATURE-----
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.
MailScanner thanks transtec Computers for their support.
More information about the MailScanner