Login | Register For Free | Help
Search for: (Advanced)

Mailing List Archive: SpamAssassin: devel

[Bug 6241] [review] mkrules does not understand newer options and 'else'

 

 

SpamAssassin devel RSS feed   Index | Next | Previous | View Threaded


bugzilla-daemon at bugzilla

Jan 27, 2010, 2:08 PM

Post #1 of 2 (342 views)
Permalink
[Bug 6241] [review] mkrules does not understand newer options and 'else'

https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6241

Justin Mason <jm [at] jmason> changed:

What |Removed |Added
----------------------------------------------------------------------------
Summary|mkrules does not understand |[review] mkrules does not
|newer options and 'else' |understand newer options
| |and 'else'
Status Whiteboard| |needs 2 votes

--- Comment #8 from Justin Mason <jm [at] jmason> 2010-01-27 14:08:31 UTC ---
checked in on trunk:

: 130...; svn commit -m "bug 6241: build/mkrules needs to support nested
conditionals and 'else'"
Sending MANIFEST
Sending build/mkrules
Sending rulesrc/sandbox/khopesh/20_khop_bl.cf
Adding t/mkrules_else.t
Transmitting file data ....
Committed revision 903860.

please vote for checkin to 3.3...

--
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.


bugzilla-daemon at bugzilla

Feb 3, 2010, 9:44 AM

Post #2 of 2 (264 views)
Permalink
[Bug 6241] [review] mkrules does not understand newer options and 'else' [In reply to]

https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6241

--- Comment #9 from Mark Martinec <Mark.Martinec [at] ijs> 2010-02-03 09:44:16 UTC ---
> please vote for checkin to 3.3...

+1

Results look alright, a major improvement!
Code changes make sense (I don't claim I fully understand
the fine details), and the added test was much needed.

--
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.

SpamAssassin devel RSS feed   Index | Next | Previous | View Threaded
 
 


Interested in having your list archived? Contact Gossamer Threads
 
  Web Applications & Managed Hosting Powered by Gossamer Threads Inc.