Bug 7308

Summary:	Include "forget:" keyword to sa-learn -f option
Product:	Spamassassin	Reporter:	apachebugs
Component:	Learner	Assignee:	SpamAssassin Developer Mailing List <dev>
Status:	NEW ---
Severity:	enhancement	CC:	apache, apachebugs, david, kmcgrail
Priority:	P2
Version:	unspecified
Target Milestone:	Future
Hardware:	All
OS:	All
Whiteboard:

Description apachebugs 2016-04-08 15:39:15 UTC

Currently, sa-learn's `-f` can be used to replace `sa-learn --ham /some/dir` and `sa-learn --spam /other/dir` by a single `sa-learn -f` call. However, it is not possible to replace a `sa-learn --forget /third/dir` the same way.

I would propose to add a keyword `forget` to the `sa-learn -f` file parser, so that a configuration like this works:


$ sa-learn -f config 
$ cat config
ham:dir:/var/mail/me/cur
spam:dir:/var/mail/me/.Junk/cur
forget:dir:/var/mail/me/.Trash/cur

Comment 1 Kevin A. McGrail 2018-08-25 23:11:16 UTC

Great idea.  Any chance you can take a pass at a patch for this idea?

Comment 2 apachebugs 2018-08-30 20:58:45 UTC

> Great idea.  Any chance you can take a pass at a patch for this idea?
Thanks for picking up this bug, but, sorry, no -- I can somewhat understand
Perl if people use simple phrases and not-so-complicated words ;) but I
don't speak it well enough to write code that real people would use...

Comment 3 Henrik Krohns 2022-04-15 08:38:35 UTC

I think it would need a little more code refactoring, probably not happening for 4.0, postponing..