http://bugzilla.spamassassin.org/show_bug.cgi?id=3064





------- Additional Comments From [EMAIL PROTECTED]  2004-03-01 08:12 -------
Subject: Re:  sa-learn should ignore "use_bayes 0" setting

On Sun, Feb 29, 2004 at 11:48:26PM -0800, [EMAIL PROTECTED] wrote:
> 
> I propose that we change the "use_bayes" option only turn off automatic
> activity: checks and auto-learning, but we should ignore the setting for
> manual operations.  I agree with Theo that the override should not be done
> in sa-learn.  It should just be part of how the option works.
> 

At first I thought, maybe a new config param, use_salearn, but I don't
think we need to complicate things.  I just did a quick run through of
all the places that check use_bayes and it's not many.  I believe we
can get rid of all of the except the ones in:
Bayes::is_scan_available
PerMsgStatus::learn
and of course the Conf.pm stuff.

This will cause use_bayes to be used for only scanning/check and
auto-learning and will allow users to use sa-learn whenever they want,
for whatever they want.

If this is agreeable, I can finish up this patch and commit.

Michael





------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.

Reply via email to