|
SA Bugzilla – Full Text Bug Listing |
Summary: | ruleqa is handling late/early masscheck data badly | ||
---|---|---|---|
Product: | Spamassassin | Reporter: | Darxus <Darxus> |
Component: | RuleQA | Assignee: | SpamAssassin Developer Mailing List <dev> |
Status: | RESOLVED FIXED | ||
Severity: | normal | CC: | kmcgrail, nick, paul.stead |
Priority: | P2 | ||
Version: | SVN Trunk (Latest Devel Version) | ||
Target Milestone: | Undefined | ||
Hardware: | All | ||
OS: | All | ||
Whiteboard: |
Description
Darxus
2012-01-31 17:59:19 UTC
Should we instead allow an SVN range +/- 1000 or something similar? If I understand the outcome of making it a range, not sure this would be wise as it could possibly not reflect changes on rule changes/revisions, etc. ,delay score changes and would be even harder to find out if someone's new rules are worth any effort while working on a rule subset *** Bug 6821 has been marked as a duplicate of this bug. *** Perhaps setting that the svn checkout or rsync for the code will utilize a the FIRST check-in for the day. That will leave us ALWAYS one day behind on code but give a 24-hour window where all mass-check clients are using the same codebase. (In reply to comment #4) > Perhaps setting that the svn checkout or rsync for the code will utilize a > the FIRST check-in for the day. That will leave us ALWAYS one day behind on > code but give a 24-hour window where all mass-check clients are using the > same codebase. not sure it the FIRST is a good plan. Assuming the first commit of the day is a code change which is borked and woudl get a fix afterwards.... I'd go for last of the day before where chances of meeting good fixes are higher. Kevin Golding is getting SVN revision: unknown I've asked him to add a few debug statements to the mass-check script to see which scenario in get_current_svn_revsion is being used. My theory is he is running the svn command line and you might have a broken version of svn. You might need to compile your own. Anyone know if it would it break things to add information to the get_current_svn_revision on stdout? I think we should add code that if revision is unknown, we abort a masscheck. It's not going to get used...
Kevin Golding : I added in extra statements outside the conditionals and it proved I'm not successfully entering any of them. It looks like my problem is line 998:
if (-d "$dir/.svn" || -f "$dir/svninfo.tmp") {
At that point $dir = /usr/home/masscheck/trunk/masses
I have a /usr/home/masscheck/trunk/.svn but no /usr/home/masscheck/trunk/masses/.svn
There is a patch in note 6821 about this but the root cause is pointed out by Kris Deugau - SVN 1.7+:
> Have you upgraded to SVN 1.7? The working copy structure uses only one
> .svn directory at the root of the working copy for 1.7, so if you've
> upgraded from < 1.7, your working copy at /usr/home/masscheck/trunk will
> not have a /usr/home/masscheck/trunk/masses/.svn directory any more.
We'll need to accommodate a case for that!
We need to look more into these files: http://rsync.spamassassin.org/nightly-versions.txt (and http://rsync.spamassassin.org/weekly-versions.txt of course) Is the masscheck script using this to determine what version to download? Can the version that some people are using with RSYNC to "checkout" grab a specific SVN version? Fixed with https://bz.apache.org/SpamAssassin/show_bug.cgi?id=7715 Early/late submissions are categorised with corpus files from the same date. Further problems with active.list exist - investigation continues. https://bz.apache.org/SpamAssassin/enter_bug.cgi |