bugzilla-daemon at bugzilla
Jun 19, 2009, 2:03 PM
Post #1 of 1
(248 views)
Permalink
|
|
[Bug 6138] New: "Hudson build is still unstable" messages
|
|
https://issues.apache.org/SpamAssassin/show_bug.cgi?id=6138
Summary: "Hudson build is still unstable" messages
Product: Spamassassin
Version: SVN Trunk (Latest Devel Version)
Platform: All
OS/Version: All
Status: NEW
Severity: major
Priority: P5
Component: Building & Packaging
AssignedTo: dev [at] spamassassin
ReportedBy: sidney [at] sidney
Created an attachment (id=4466)
--> (https://issues.apache.org/SpamAssassin/attachment.cgi?id=4466)
Proposed patch
The Hudson builds started producing error messages reporting the build as
"unstable" for every build starting around June 12, 2009. For example see
http://hudson.zones.apache.org/hudson/job/SpamAssassin-trunk/3291/changes
There has been quite a bit of discussion on sa-dev mailing list. My analysis is
that it is caused by the loadplugin in v330.pre that uses the optional filename
syntax of loadplugin, which specifies to load the file from the current
directory. SATest.pm copies all the *.pre files from t/log/test_rules_copy/ to
t/log/localrules.tmp/. That causes the loadplugin to fail because the file,
EmailBL.pm is not also copied there. This all results from using a test rule
(EmailBL.PM) that is a plugin, something that I don't see having been done
before. Looking at old logs from Hudson, I see that the error in the build
started happening when EmailBL was first added back around May 11 or 12.
Something changed in Hudson that made it noisier and reporting the already
existing error on June 12.
I think that the correct fix is to copy *.pm at the same time that *.pre is
copied to t/log/localrules.tmp/ because any such loadplugin in a pre file
requires that the associated pm file be in the same directory.
--
Configure bugmail: https://issues.apache.org/SpamAssassin/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the assignee for the bug.
|
