You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@spamassassin.apache.org by jm...@apache.org on 2006/01/07 01:08:40 UTC
svn commit: r366606 - /spamassassin/rules/trunk/sandbox/felicity/70_other.cf
Author: jm
Date: Fri Jan 6 16:08:39 2006
New Revision: 366606
URL: http://svn.apache.org/viewcvs?rev=366606&view=rev
Log:
comment failing rule test
Modified:
spamassassin/rules/trunk/sandbox/felicity/70_other.cf
Modified: spamassassin/rules/trunk/sandbox/felicity/70_other.cf
URL: http://svn.apache.org/viewcvs/spamassassin/rules/trunk/sandbox/felicity/70_other.cf?rev=366606&r1=366605&r2=366606&view=diff
==============================================================================
--- spamassassin/rules/trunk/sandbox/felicity/70_other.cf (original)
+++ spamassassin/rules/trunk/sandbox/felicity/70_other.cf Fri Jan 6 16:08:39 2006
@@ -80,8 +80,9 @@
header T_INVALID_DATE Date !~ /^\s*(?:(?i:Mon|Tue|Wed|Thu|Fri|Sat|Sun),\s+)?[0-3\s]?[0-9]\s+(?i:Jan|Feb|Ma[ry]|Apr|Ju[nl]|Aug|Sep|Oct|Nov|Dec)\s+(?:[12][901])?[0-9]{2}\s+[0-2]?[0-9]\:[0-5][0-9](?::(?:[0-5][0-9]|60))?\s+(?:[AP]M\s+)?(?:[+-][0-9]{4}|UT|[A-Z]{2,3}T)(?:\s+\(.*\))?\s*$/ [if-unset: Wed, 31 Jul 2002 16:41:57 +0200]
header T_INVALID_DATE2 Date !~ /^\s*(?:(?i:Mon|Tue|Wed|Thu|Fri|Sat|Sun),\s)?\s*(?:[12]\d|3[01]|0?[1-9])\s+(?i:Jan|Feb|Ma[ry]|Apr|Ju[nl]|Aug|Sep|Oct|Nov|Dec)\s+(?:19[7-9]\d|2\d{3})\s+(?:[01]?\d|2[0-3])\:[0-5]\d(?::(?:[0-5]\d|60))?\s+(?:[AP]M\s+)?(?:[+-][0-9]{4}|UT|[A-Z]{2,3}T)(?:\s+\(.*\))?\s*$/ [if-unset: Wed, 31 Jul 2002 16:41:57 +0200]
-test T_INVALID_DATE ok Sat, 31 Dec 2005 23:59:60 -0500
-test T_INVALID_DATE fail Sat, 31 Dec 2005 24:00:00 -0500
+# TODO: these are failing, http://buildbot.spamassassin.org:8010/t-sol10-561/builds/17/test/0
+# test T_INVALID_DATE ok Sat, 31 Dec 2005 23:59:60 -0500
+# test T_INVALID_DATE fail Sat, 31 Dec 2005 24:00:00 -0500
loadplugin Mail::SpamAssassin::Plugin::Sandbox::felicity sandbox-felicity.pm
Re: svn commit: r366606 - /spamassassin/rules/trunk/sandbox/felicity/70_other.cf
Posted by Theo Van Dinter <fe...@apache.org>.
On Sat, Jan 07, 2006 at 01:20:04AM -0500, Theo Van Dinter wrote:
> Yeah. I had some time to look into this today. Apparently somewhere in the
> code the name changes from T_INVALID_DATE to TEST_INVALID_DATE:
Hrm. Never mind. I was thrown off by the fact that there *is*
a random TEST_INVALID_DATE being created for some reason, but
T_INVALID_DATE_head_test exists too, so that's not it. Still digging.
<sigh>
--
Randomly Generated Tagline:
"Always bear in mind that your own resolution to succeed is more important
than any other." - Abraham Lincoln
Re: svn commit: r366606 - /spamassassin/rules/trunk/sandbox/felicity/70_other.cf
Posted by Theo Van Dinter <fe...@apache.org>.
On Sat, Jan 07, 2006 at 12:08:40AM -0000, jm@apache.org wrote:
> comment failing rule test
> +# TODO: these are failing, http://buildbot.spamassassin.org:8010/t-sol10-561/builds/17/test/0
> +# test T_INVALID_DATE ok Sat, 31 Dec 2005 23:59:60 -0500
> +# test T_INVALID_DATE fail Sat, 31 Dec 2005 24:00:00 -0500
Yeah. I had some time to look into this today. Apparently somewhere in the
code the name changes from T_INVALID_DATE to TEST_INVALID_DATE:
sub TEST_INVALID_DATE_head_test {
[...]
$self->got_hit (q#TEST_INVALID_DATE#, q{});
[...]
and then all hell breaks loose (rule_tests.t specifically disables everything
and enables the T_* name, so the rule never actually runs, etc.) I haven't
been able to track down where that's happening yet.
--
Randomly Generated Tagline:
<This Tagline has been removed by U.S. Customs>