Closed
Bug 919635
Opened 10 years ago
Closed 10 years ago
Support Steeplechase tests in tree
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla27
People
(Reporter: ted, Assigned: ted)
References
(Blocks 1 open bug)
Details
Attachments
(2 files)
3.40 KB,
patch
|
gps
:
review+
|
Details | Diff | Splinter Review |
2.54 KB,
patch
|
gps
:
review+
|
Details | Diff | Splinter Review |
For the new Steeplechase test harness I'm writing we want test manifests. This patch builds on the patch from bug 901990 to add test manifests for Steeplechase. Fairly simple patch.
Assignee | ||
Comment 1•10 years ago
|
||
Attachment #808686 -
Flags: review?(gps)
Assignee | ||
Updated•10 years ago
|
Blocks: steeplechase
Depends on: 901990
Assignee | ||
Comment 2•10 years ago
|
||
The next step here is to package these tests into the test package.
Comment 3•10 years ago
|
||
Comment on attachment 808686 [details] [diff] [review] Add manifests for Steeplechase tests to moz.build Review of attachment 808686 [details] [diff] [review]: ----------------------------------------------------------------- This is almost too simple.
Attachment #808686 -
Flags: review?(gps) → review+
Assignee | ||
Comment 4•10 years ago
|
||
I hate test packaging. I think I'm going to move that all into moz.build soon. In the interest of getting this landed I'll go with this instead.
Attachment #809311 -
Flags: review?(gps)
Comment 5•10 years ago
|
||
Comment on attachment 809311 [details] [diff] [review] package Steeplechase tests and extra bits in test package Review of attachment 809311 [details] [diff] [review]: ----------------------------------------------------------------- I can't wait for this make file to die.
Attachment #809311 -
Flags: review?(gps) → review+
Assignee | ||
Comment 6•10 years ago
|
||
remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/5b1a56afe153 remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/0fae05649fe8
Comment 7•10 years ago
|
||
Backed out for: https://tbpl.mozilla.org/php/getParsedLog.php?id=28341189&tree=Mozilla-Inbound remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/1afd02a98cef remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/3408740409b3
Assignee | ||
Comment 8•10 years ago
|
||
Once more with feeling (and a one-line whitespace docstring comment fix to fix that pedantic unit test): remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/162e33423f59 remote: https://hg.mozilla.org/integration/mozilla-inbound/rev/f4087a24f020
https://hg.mozilla.org/mozilla-central/rev/162e33423f59 https://hg.mozilla.org/mozilla-central/rev/f4087a24f020
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla27
Updated•5 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•