Closed
Bug 1179050
Opened 9 years ago
Closed 9 years ago
Add spidermonkey try flags to TryChooser
Categories
(Release Engineering :: General, defect)
Release Engineering
General
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: ffledgling, Assigned: ffledgling)
Details
Attachments
(1 file)
3.42 KB,
patch
|
dustin
:
review+
|
Details | Diff | Splinter Review |
Spidermonkey builds are now being added as 1st class citizens to try (so to speak), and have their own try flags. This should be reflected in the TryChooser Syntax
Assignee | ||
Comment 1•9 years ago
|
||
Attachment #8628025 -
Flags: review?(dustin)
Assignee | ||
Comment 2•9 years ago
|
||
Important note, Landing this needs to wait till the patch for Bug 1174376 is landed.
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → ffledgling
Comment 3•9 years ago
|
||
Comment on attachment 8628025 [details] [diff] [review] Patch to add flags Review of attachment 8628025 [details] [diff] [review]: ----------------------------------------------------------------- Looks good, assuming the generated syntax is what you would like.
Attachment #8628025 -
Flags: review?(dustin) → review+
Assignee | ||
Comment 4•9 years ago
|
||
Yup, that's the syntax, used it to trigged this push -- https://treeherder.mozilla.org/#/jobs?repo=try&revision=cd92907122d0 It worked as expected.
Assignee | ||
Comment 5•9 years ago
|
||
Check-In needed here, since the other two bugs that needed to land before it have landed. NI'ing catlee since I didn't find a checkin? flag.
Assignee | ||
Updated•9 years ago
|
Flags: needinfo?(catlee)
Assignee | ||
Comment 6•9 years ago
|
||
Pushed -- http://hg.mozilla.org/build/tools/rev/3457fd906a2c
Status: NEW → RESOLVED
Closed: 9 years ago
Flags: needinfo?(catlee)
Resolution: --- → FIXED
Assignee | ||
Comment 7•9 years ago
|
||
Changes live @ http://trychooser.pub.build.mozilla.org/ thanks to nthomas.
Updated•6 years ago
|
Component: General Automation → General
You need to log in
before you can comment on or make changes to this bug.
Description
•