Closed Bug 527015 Opened 10 years ago Closed 10 years ago

Clean up custom build scripts for running debug/debug-debugger acceptance

Categories

(Tamarin Graveyard :: Build Config, defect)

defect
Not set

Tracking

(Not tracked)

VERIFIED FIXED
flash10.1

People

(Reporter: brbaker, Assigned: brbaker)

References

Details

Attachments

(1 file)

Currently there are custom build scripts for each build slave to run acceptance tests for the debug and debug-debugger avmshells. This was necessary as previously there was no easy way to tell if a shell was a debug build or not. Now with bug# 526871 the runtest.py is now able to figure that out.

Should remove the unnecessary build/buildbot/slaves/*/scripts/run-acceptance-debug* scripts and update the common script to work for all slaves. Will also need to update the buildbot/master/master.cfg for the build step.
Flags: flashplayer-triage+
Flags: flashplayer-qrb?
Assignee: nobody → brbaker
Status: NEW → ASSIGNED
Target Milestone: --- → flash10.1
Flags: flashplayer-qrb? → flashplayer-qrb+
Update the running of all of the acceptance runs to use the run-acceptance-generic script.

By removing the supporting scripts that used to define how and what was run, and instead keeping all of the information in the configuration file it will allow us to parse the data and report on what is being tested.

Still to figure out is how to properly handle the windows-mobile running of acceptance.
Attachment #418387 - Flags: review?(cpeyer)
Pushed changeset: 3415:84209df4acbe
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Status: RESOLVED → VERIFIED
changeset:   3423:073f60a7be04
tag:         tip
user:        Brent Baker <brbaker@adobe.com>
date:        Mon Jan 04 13:20:00 2010 -0500
files:       build/buildbot/master/master.cfg
description:
Bug 527015: fixed typo in mac-ppc deep build slave, shell name is *_64_ppc not *_ppc_64 (r=self)
Comment on attachment 418387 [details] [diff] [review]
Acceptance testing

cpeyer +'ed the patch via IM
Attachment #418387 - Flags: review?(cpeyer) → review+
You need to log in before you can comment on or make changes to this bug.