[Bug 3049] Implement sa-learn --loaddb dumpfile functionality

[bu...@bugzilla.spamassassin.org]

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





------- Additional Comments From parkerm@pobox.com  2004-02-15 17:50 -------
Subject: Re:  New: Implement sa-learn --loaddb dumpfile functionality

On Sun, Feb 15, 2004 at 05:39:49PM -0800, bugzilla-daemon@bugzilla.spamassassin.org wrote:
> 
> It would be really handy to be able to populate a bayes database file from a
> dump file.  This would allow folks to dump a database, edit it and then re-load it.
> 

I've got a good portion of this done and tested, but have a few
questions/design decisions I want to make sure are the right thing:

1) Right now it creates a whole new database file and instructs the
   user to copy it into place to start using it.  Should it go ahead
   and do the copy for them? or let the user do it just in case?

2) The few folks recently that needed this functionality had ham/spam
   counts < 0, this isn't right correct?  One of the things this does
   is sets any negative counts back to 0.

3) Related to number 2, folks have atimes way in the future, if the
   atime is > time() then I go ahead and set it to time().

4) If not database version 2 then it won't work.

5) All of the required "magic" variables must be found in the dumpfile
   otherwise it is considered an error and bails out.

6) sa-learn isn't ARGV friendly it seems, so the dumpfile name has to
   be part of the --loaddb argument.

Like I said, I've pretty much got a patch ready to go for this, just
not certain on a few of the decisions I made.

Michael





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