Closed Bug 1179050 Opened 7 years ago Closed 7 years ago

Add spidermonkey try flags to TryChooser

Categories

(Release Engineering :: General, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: ffledgling, Assigned: ffledgling)

Details

Attachments

(1 file)

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
Attachment #8628025 - Flags: review?(dustin)
Important note, Landing this needs to wait till the patch for Bug 1174376 is landed.
Assignee: nobody → ffledgling
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+
Yup, that's the syntax, used it to trigged this push -- https://treeherder.mozilla.org/#/jobs?repo=try&revision=cd92907122d0 It worked as expected.
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.
Flags: needinfo?(catlee)
Pushed -- http://hg.mozilla.org/build/tools/rev/3457fd906a2c
Status: NEW → RESOLVED
Closed: 7 years ago
Flags: needinfo?(catlee)
Resolution: --- → FIXED
Changes live @ http://trychooser.pub.build.mozilla.org/ thanks to nthomas.
Component: General Automation → General
You need to log in before you can comment on or make changes to this bug.