Closed
Bug 1243231
Opened 5 years ago
Closed 5 years ago
Implement a taskcluster-based b2g hazard build
Categories
(Core :: JavaScript: GC, defect)
Core
JavaScript: GC
Tracking
()
RESOLVED
FIXED
mozilla47
Tracking | Status | |
---|---|---|
firefox47 | --- | fixed |
People
(Reporter: sfink, Assigned: sfink)
References
Details
Attachments
(2 files)
7.25 KB,
patch
|
garndt
:
review+
terrence
:
review+
|
Details | Diff | Splinter Review |
3.71 KB,
patch
|
Details | Diff | Splinter Review |
The buildbot builds are all going away (see bug 1236835). The hazard build is orphaned. We need(ed) a new taskcluster-based hazard build. Except that b2g is tier 3 now, so we don't really. But I have this working, it is much simpler, and it will serve as a good base for a taskcluster browser hazard build.
Assignee | ||
Comment 1•5 years ago
|
||
The only issue with this is that it finds a hazard. :( But it's the same paramBuffer false positive that I battle with off and on.
Assignee | ||
Comment 2•5 years ago
|
||
Comment on attachment 8712450 [details] [diff] [review] Implement a taskcluster-based b2g hazard build Review of attachment 8712450 [details] [diff] [review]: ----------------------------------------------------------------- Terrence - can you review the portions of the shell script (build-mulet-haz-linux.sh, build.b2g, and the mulet-hazards mozconfig) that have to do with running the analysis? They're happily quite dull. Greg, all of the job setup is for you. There are some strangenesses I inflicted on shared code, I'll comment on those sections. I suppose I could have split this into two patches with an empty body for build-mulet-haz-linux.sh... ::: testing/taskcluster/scripts/builder/install-packages.sh @@ +8,5 @@ > test -n "$TOOLTOOL_REV" > > tc-vcs checkout $gecko_dir/tooltool $TOOLTOOL_REPO $TOOLTOOL_REPO $TOOLTOOL_REV > > +(cd $gecko_dir; python $gecko_dir/tooltool/tooltool.py --url https://api.pub.build.mozilla.org/tooltool/ -m $gecko_dir/$TOOLTOOL_MANIFEST fetch -c $TOOLTOOL_CACHE) I removed --overwrite because it's deprecated, and I have another patch where I use this to cache tooltool checkouts (not just downloads; that's already cached). It helps with the development debug cycle because the rustc checkout takes quite a while. Probably, though, it'll need to be opt-in since it sticks a file into the checkout. Anyway, for now this is just a removal of a deprecated option.
Attachment #8712450 -
Flags: review?(terrence)
Attachment #8712450 -
Flags: review?(garndt)
Updated•5 years ago
|
Attachment #8712450 -
Flags: review?(terrence) → review+
Updated•5 years ago
|
Attachment #8712450 -
Flags: review?(garndt) → review+
Comment 4•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/3d625cdac138
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
status-firefox47:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla47
Assignee | ||
Comment 5•5 years ago
|
||
I split out enabling this on main trees, starting out with just having it on try, until I disabled the original b2g hazard job. That has now been done, though currently it's awaiting the next reconfigure, so it's time to push this portion too.
Comment 7•5 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/eb0d3da74f19
You need to log in
before you can comment on or make changes to this bug.
Description
•