https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7735

Henrik Krohns <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |---

--- Comment #76 from Henrik Krohns <[email protected]> ---
(In reply to Wolfgang Breyha from comment #75)
> I checked out rev 1905124 today, built a local package and tried to test it
> with some of my test E-Mails I used before.

Thanks for testing, it's hard to make comprehensive tests for this..

I think it might be as simple as this:

--- lib/Mail/SpamAssassin/Plugin/Check.pm       (revision 1905124)
+++ lib/Mail/SpamAssassin/Plugin/Check.pm       (working copy)
@@ -316,7 +316,7 @@
       $pms->got_hit($rulename, '', ruletype => 'meta', value => $result);
     } else {
       dbg("rules-all: ran meta rule $rulename, no hit") if
$would_log_rules_all;
-      $h->{$rulename} = 0; # mark meta done
+      $pms->rule_ready($rulename, 1); # mark meta done
     }
     delete $mr->{$rulename};
     delete $mp->{$rulename};

Can you try 1905124 + that change? rule_ready() does the same thing as your
patch. Also got_hit() calls rule_ready(), in case someone is wondering.

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

Reply via email to