http://bugzilla.spamassassin.org/show_bug.cgi?id=3049
------- Additional Comments From [EMAIL PROTECTED] 2004-02-15 17:50 ------- Subject: Re: New: Implement sa-learn --loaddb dumpfile functionality On Sun, Feb 15, 2004 at 05:39:49PM -0800, [EMAIL PROTECTED] 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.
