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





------- Additional Comments From [EMAIL PROTECTED]  2004-03-01 11:15 -------
> you're view is a little bit off here.  "use_bayes" tells the SA code
> to not use the Bayes code, any Bayes DBs, etc.  It's an on/off flag
> for the whole thing.  As part of that of course, rules and
> autolearning are disabled since the whole thing is being disabled.

I was talking about how and why *users* use the option and *not* how it
works in the code.  They are two different things, as evidenced by the
amount of confusion and consternation that the current implementation
causes.

> Well, the other problem with this, of course, is that this is great
> for us, but what about the people who use our modules directly?  I,
> for instance, have code that calls M::SA->learn() directly.  Should
> that work if the config says "use_bayes 0"?  IMO, it's up to the
> modules to check, not my calling code.

I agree that the configuration should be what matters, but I think
having separate options for learning and checking (at least) would be
useful and a major improvement.  Nobody has recently suggested that we
have any command-line options override these settings.

> So far we're -1.9 and +2, fyi.  I'm leaning more to a -1 now, actually.
> So make that -2 and +2 overall.

Note that any -1 is a veto for a code modification.  I don't think we're
at the point where voting is meaningful yet.




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

Reply via email to