Closed
Bug 490620
Opened 16 years ago
Closed 16 years ago
Buildbot acceptance scripts should confirm binaries are available
Categories
(Tamarin Graveyard :: Build Config, defect)
Tracking
(Not tracked)
VERIFIED
FIXED
People
(Reporter: brbaker, Assigned: brbaker)
Details
Attachments
(1 file)
32.18 KB,
patch
|
dschaffe
:
review+
|
Details | Diff | Splinter Review |
Currently all of the acceptance buildbot scripts blindly assume that the shell binaries are available on the machine. This is ok at the moment because all of the acceptance testing is happening on the same machine that built the shell. However we want to move some of the compilation off of slower machines, but will still need to run acceptance on the slow machines.
All of the run-acceptance/performace scripts should
* actually only download the asc.jar if it does not already exist
* download the necessary shell binary if it doesn't already exist
* These steps should be moved into a common base run-acceptance script and duplication removed
Flags: flashplayer-qrb?
Assignee | ||
Comment 1•16 years ago
|
||
If the avmshell or asc.jar are not available locally, download them.
Attachment #375051 -
Flags: review?(dschaffe)
Updated•16 years ago
|
Attachment #375051 -
Flags: review?(dschaffe) → review+
Comment 2•16 years ago
|
||
Comment on attachment 375051 [details] [diff] [review]
verify binaries are available
locally I ran run-acceptance-release.sh with and without the binaries in the builds directory. worked ok.
Assignee | ||
Comment 3•16 years ago
|
||
Pushed as
changeset: 1810:11046b7f7b01
tag: tip
user: Brent Baker <brbaker@adobe.com>
date: Wed Apr 29 14:53:27 2009 -0400
description:
Bug 490620: Buildbot acceptance scripts should confirm binaries are available. (
r=dschaffe)
Bug 490521: enable email notification in the build system
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•16 years ago
|
Status: RESOLVED → VERIFIED
Assignee | ||
Comment 4•16 years ago
|
||
removing QRB request, bug resolved/verified
Flags: flashplayer-qrb?
You need to log in
before you can comment on or make changes to this bug.
Description
•