You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@spamassassin.apache.org by bu...@bugzilla.spamassassin.org on 2005/07/13 03:27:10 UTC

[Bug 4480] New: test failure in pre4 candidate due to meta.t bug

http://bugzilla.spamassassin.org/show_bug.cgi?id=4480

           Summary: test failure in pre4 candidate due to meta.t bug
           Product: Spamassassin
           Version: SVN Trunk (Latest Devel Version)
          Platform: Other
        OS/Version: other
            Status: NEW
          Severity: blocker
          Priority: P5
         Component: Regression Tests
        AssignedTo: dev@spamassassin.apache.org
        ReportedBy: jm@jmason.org


PYZOR_CHECK was given a non-zero score in scoresets 0 and 2.  this had the
unfortunate side-effect of triggering a bug in meta.t -- it didn't like the idea
that some of the predicates (but not all) had zero scores, so issued a test failure.

this is causing test failures in the pre4 candidate tarballs -- but not on all
platforms, since meta.t relies on being able to run parse-rules-for-masses.  So
only if you have the Storable module installed. urgh.

the fix is for meta.t to allow predicate rules to have zero scores in some
scoresets if the rules are "tflags net" or "tflags learn".

patch to follow.



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

[Bug 4480] test failure in pre4 candidate due to meta.t bug

Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4480


jm@jmason.org changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|Undefined                   |3.1.0




------- Additional Comments From jm@jmason.org  2005-07-12 18:29 -------
this may block the 3.1.0-pre4 candidate I just posted.

we need to decide if we want to cut a new tarball to fix this test-suite bug, or
just ignore the bug since it doesn't effect the actual workings of the product
(it's a test suite bug only).  I vote the latter. ;)

the fix should be applied to svn eventually anyway, of course.



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

[Bug 4480] test failure in pre4 candidate due to meta.t bug

Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4480





------- Additional Comments From jm@jmason.org  2005-07-12 18:27 -------
Created an attachment (id=3022)
 --> (http://bugzilla.spamassassin.org/attachment.cgi?id=3022&action=view)
drat.  patch

here's the fix.



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

[Bug 4480] test failure in pre4 candidate due to meta.t bug

Posted by bu...@bugzilla.spamassassin.org.
http://bugzilla.spamassassin.org/show_bug.cgi?id=4480


jm@jmason.org changed:

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




------- Additional Comments From jm@jmason.org  2005-07-13 11:16 -------
checked it in as a trivial fix, since it's the addition of one statement (and
about 5 lines of explanatory comments) in a t script, revision 216202.




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