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 2007/11/29 11:00:52 UTC
[Bug 5737] New: t/basic_meta.t failing
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5737
Summary: t/basic_meta.t failing
Product: Spamassassin
Version: SVN Trunk (Latest Devel Version)
Platform: Other
OS/Version: other
Status: NEW
Severity: normal
Priority: P5
Component: Rules
AssignedTo: dev@spamassassin.apache.org
ReportedBy: jm@jmason.org
t/basic_meta....................DOS_OUTLOOK_TO_MX depends on
DOS_OUTLOOK_TO_MX_IMAGE with 0 score in set 2
DOS_OE_TO_MX depends on DOS_OE_TO_MX_IMAGE with 0 score in set 2
DOS_OUTLOOK_TO_MX depends on DOS_OUTLOOK_TO_MX_IMAGE with 0 score in set 3
DOS_OE_TO_MX depends on DOS_OE_TO_MX_IMAGE with 0 score in set 3
# Failed test 1 in t/basic_meta.t at line 90
FAILED test 1
Failed 1/2 tests, 50.00% okay
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5737] t/basic_meta.t failed due to race condition between automated daily checkins
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5737
jm@jmason.org changed:
What |Removed |Added
----------------------------------------------------------------------------
Summary|t/basic_meta.t failing |t/basic_meta.t failed due to
| |race condition between
| |automated daily checkins
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5737] t/basic_meta.t failing
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5737
jm@jmason.org changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|RESOLVED |REOPENED
Resolution|FIXED |
------- Additional Comments From jm@jmason.org 2007-11-30 02:08 -------
(In reply to comment #3)
> build/mkrules was using the generated scores... the generated scores aren't
> always in sync with the active.list file (the scores are generated after that
> file is checked in, so there's always at least one revision where they're out of
> sync)... get the right combination of rules dropping in and out of the
> active.list and you get meta dependency issues.
Wait a second -- this change means that the scores in
rulesrc/scores/72_scores.cf are now unused. They don't make it into the "rules"
dir output at all :(
That's not good -- we want those scores to be published and active!
I can change the timing of active.list generation, would that help?
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5737] t/basic_meta.t failing
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5737
spamassassin@dostech.ca changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|Undefined |3.3.0
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5737] t/basic_meta.t failing
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5737
------- Additional Comments From jm@jmason.org 2007-11-30 12:09 -------
(In reply to comment #5)
> I think the way to go is to have mkrules only use the scores if they were
> generated after the active.list file was generated.
that would work (and would be easy enough, too) if we can just go by mod times
(which I think we can)
> I'll move the score generation to just after the active.list is generated (I've
> been meaning to do that anyway... there's no need for the lag, 7pm was just
> convenient at the time I wrote it (my dialup line was idle enough to download
> the logs). Then we'll just have to delay the generation of the update tarball
> by about an hour (or get it to wait for the new scores to be committed and then
> proceed).
cool.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5737] t/basic_meta.t failing
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5737
------- Additional Comments From spamassassin@dostech.ca 2007-11-30 09:54 -------
Oops, brain fart. For some reason I was thinking that the updates would still
have the scores, but that makes no sense since they're made with build/mkrules
and they're made out of sync too.
I think the way to go is to have mkrules only use the scores if they were
generated after the active.list file was generated.
I'll move the score generation to just after the active.list is generated (I've
been meaning to do that anyway... there's no need for the lag, 7pm was just
convenient at the time I wrote it (my dialup line was idle enough to download
the logs). Then we'll just have to delay the generation of the update tarball
by about an hour (or get it to wait for the new scores to be committed and then
proceed).
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5737] t/basic_meta.t failing
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5737
spamassassin@dostech.ca changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |RESOLVED
Resolution| |FIXED
------- Additional Comments From spamassassin@dostech.ca 2007-11-29 17:57 -------
build/mkrules was using the generated scores... the generated scores aren't
always in sync with the active.list file (the scores are generated after that
file is checked in, so there's always at least one revision where they're out of
sync)... get the right combination of rules dropping in and out of the
active.list and you get meta dependency issues.
[dos@cyan trunk]$ svn ci -m "bug 5737: mkrules cannot use generated scores since
the generated scores are not guaranteed to be in sync with the current
active.list file"
Sending build/mkrules
Transmitting file data .
Committed revision 599674.
[dos@cyan trunk]$
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5737] t/basic_meta.t failing
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5737
------- Additional Comments From jm@jmason.org 2007-11-29 08:13 -------
it was
http://buildbot.spamassassin.org:8010/t-solaris-10/builds/1521/step-test/0 , so
599346.
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.
[Bug 5737] t/basic_meta.t failing
Posted by bu...@bugzilla.spamassassin.org.
http://issues.apache.org/SpamAssassin/show_bug.cgi?id=5737
------- Additional Comments From spamassassin@dostech.ca 2007-11-29 08:05 -------
What revision was this run on?
------- You are receiving this mail because: -------
You are the assignee for the bug, or are watching the assignee.