Closed Bug 412807 Opened 16 years ago Closed 16 years ago

integrate buildbot config file tester w/ buildbot

Categories

(Release Engineering :: General, defect, P2)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: bhearsum, Assigned: bhearsum)

Details

Attachments

(2 files, 1 obsolete file)

Attached patch integrate config file tester (obsolete) — Splinter Review
This patch integrates a config file tester into Buildbot. It makes it accessible through 'buildbot checkconfig'. I've already posted this on buildbot.net in ticket#37: http://buildbot.net/trac/ticket/37
Attachment #297570 - Flags: review?(rcampbell)
I finally ran the unit tests on this patch and discovered an indentation error. All of the unit tests pass now.
Attachment #297570 - Attachment is obsolete: true
Attachment #297762 - Flags: review?(rcampbell)
Attachment #297570 - Flags: review?(rcampbell)
Priority: -- → P2
Comment on attachment 297762 [details] [diff] [review]
[checked in] fix indentation error

looks good.
Attachment #297762 - Flags: review?(rcampbell) → review+
Comment on attachment 297762 [details] [diff] [review]
[checked in] fix indentation error

Checking in buildbot/scripts/runner.py;
/cvsroot/mozilla/tools/buildbot/buildbot/scripts/runner.py,v  <--  runner.py
new revision: 1.2; previous revision: 1.1
done
Attachment #297762 - Attachment description: fix indentation error → [checked in] fix indentation error
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
I've just noticed that this patch _requires_ Python 2.5 ('finally' doesn't exist in previous versions). I'll have a patch up soon that supports at least Python 2.4 and Python 2.5.
Status: RESOLVED → REOPENED
Resolution: FIXED → ---
Attachment #300013 - Flags: review?(rcampbell) → review+
Comment on attachment 300013 [details] [diff] [review]
[checked in] call chdir/rmtree in except and outside

Checking in buildbot/scripts/runner.py;
/cvsroot/mozilla/tools/buildbot/buildbot/scripts/runner.py,v  <--  runner.py
new revision: 1.2.2.1; previous revision: 1.2
done
Attachment #300013 - Attachment description: call chdir/rmtree in except and outside → [checked in] call chdir/rmtree in except and outside
Status: REOPENED → RESOLVED
Closed: 16 years ago16 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: