Bug 5514 - [review] make "score set for non-existent rule" a debug message
Summary: [review] make "score set for non-existent rule" a debug message
Status: RESOLVED FIXED
Alias: None
Product: Spamassassin
Classification: Unclassified
Component: Libraries (show other bugs)
Version: 3.2.1
Hardware: Other other
: P5 minor
Target Milestone: 3.2.4
Assignee: SpamAssassin Developer Mailing List
URL:
Whiteboard: ready to commit
Keywords:
Depends on:
Blocks:
 
Reported: 2007-06-13 09:38 UTC by Justin Mason
Modified: 2007-12-16 13:22 UTC (History)
0 users


Attachment Type Modified Status Actions Submitter/CLA Status
fix patch None Justin Mason [HasCLA]
Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Justin Mason 2007-06-13 09:38:34 UTC 
I suggest that this warning/error causes too much confusion; it seems
to be a very common topic on the users list, when a ruleset changes
slightly and an old customisation becomes obsolete, it causes a lint
failure.  But it's a really trivial error.

check it out, there's loads:

  http://www.google.com/search?q=%22score+set+for+non-existent+rule%22

Accordingly I suggest we "downgrade" it from a lint error to a debug
message, only visible with "-D".
Comment 1 Justin Mason 2007-12-12 15:31:52 UTC 
Created attachment 4208 [details]
fix

applied to trunk:

: jm 20...; svn commit -m "bug 5514: make 'score set for non-existent rule' a
debug message, instead of a lint warning, since it's a very frequent FAQ"
lib/Mail/SpamAssassin/Conf/Parser.pm
Sending        lib/Mail/SpamAssassin/Conf/Parser.pm
Transmitting file data .
Committed revision 603778.
Comment 2 Matt Kettler 2007-12-13 04:28:13 UTC 
+1
Comment 3 Theo Van Dinter 2007-12-16 12:44:32 UTC 
sure, +1
Comment 4 Justin Mason 2007-12-16 13:22:35 UTC 
thanks, in 3.2.x as r604708