Closed
Bug 1146520
Opened 9 years ago
Closed 8 years ago
Add OS X shell builds to Treeherder
Categories
(Core :: JavaScript Engine, defect)
Core
JavaScript Engine
Tracking
()
RESOLVED
FIXED
People
(Reporter: jandem, Assigned: sfink)
References
(Blocks 1 open bug)
Details
Attachments
(5 files)
1.91 KB,
patch
|
sfink
:
review+
|
Details | Diff | Splinter Review |
2.04 KB,
patch
|
bhearsum
:
review+
|
Details | Diff | Splinter Review |
2.26 KB,
patch
|
bhearsum
:
review+
sfink
:
checkin+
|
Details | Diff | Splinter Review |
1.50 KB,
patch
|
bhearsum
:
review+
sfink
:
checkin+
|
Details | Diff | Splinter Review |
1.66 KB,
patch
|
terrence
:
review+
sfink
:
checkin+
|
Details | Diff | Splinter Review |
We have Linux and Windows shell builds, would be great to have OS X shell builds too to avoid Mac-only bustage like bug 1146267. This patch fixes some minor autospider issues I ran into locally, but to get it working in automation we'll probably need more magic that I don't understand :)
Attachment #8581847 -
Flags: review?(sphink)
Assignee | ||
Updated•9 years ago
|
Attachment #8581847 -
Flags: review?(sphink) → review+
Assignee | ||
Updated•9 years ago
|
Keywords: leave-open
Assignee | ||
Comment 1•9 years ago
|
||
Add them into buildbot-configs.
Attachment #8581880 -
Flags: review?(bhearsum)
Assignee | ||
Updated•9 years ago
|
Assignee: nobody → sphink
Status: NEW → ASSIGNED
Comment 2•9 years ago
|
||
Comment on attachment 8581880 [details] [diff] [review] Add ARM simulator spidermonkey builds on OSX Review of attachment 8581880 [details] [diff] [review]: ----------------------------------------------------------------- stampy stamp stamp stamp
Attachment #8581880 -
Flags: review?(bhearsum) → review+
Reporter | ||
Comment 3•9 years ago
|
||
autospider patch: https://hg.mozilla.org/integration/mozilla-inbound/rev/f58f1d2a1766
Assignee | ||
Comment 4•9 years ago
|
||
https://hg.mozilla.org/build/buildbot-configs/rev/07b5232169d9
Keywords: leave-open
Comment 5•9 years ago
|
||
In production: https://hg.mozilla.org/build/buildbot-configs/rev/07b5232169d9
Comment 6•9 years ago
|
||
And permafailing, sweet. https://treeherder.mozilla.org/logviewer.html#?job_id=8043617&repo=mozilla-inbound
Comment 7•9 years ago
|
||
And hidden. Hopefully this is fixed ASAP so we're not wasting limited build slave resources on a permafailing job.
Assignee | ||
Comment 8•9 years ago
|
||
OSX builds removed for now with https://hg.mozilla.org/build/buildbot-configs/rev/2f15edda43e5 They require clang, which needs to come from tooltool. But right now, the shell builds only pull from tooltool for Windows builds. I see no reason not to use the releng manifest for all platforms at this point.
Attachment #8583231 -
Flags: review?(bhearsum)
Assignee | ||
Comment 9•9 years ago
|
||
And here's a followup I'm much less sure of. I'm activating this for both linux and osx because I can test it on my linux slave. I don't have an osx slave to test. I won't push this until it at least works on the slave.
Attachment #8583248 -
Flags: review?(bhearsum)
Assignee | ||
Updated•9 years ago
|
Keywords: leave-open
Updated•9 years ago
|
Attachment #8583231 -
Flags: review?(bhearsum) → review+
Updated•9 years ago
|
Attachment #8583248 -
Flags: review?(bhearsum) → review+
Comment 10•9 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/f58f1d2a1766 https://hg.mozilla.org/mozilla-central/rev/d907db0b4590
Assignee | ||
Comment 11•9 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/d907db0b4590
Comment 12•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-release/rev/620cae899342 https://hg.mozilla.org/releases/mozilla-release/rev/a5013ed3d1f0
status-firefox38:
--- → fixed
Updated•9 years ago
|
status-firefox38:
fixed → ---
Comment 13•9 years ago
|
||
https://hg.mozilla.org/releases/mozilla-beta/rev/620cae899342 https://hg.mozilla.org/releases/mozilla-beta/rev/a5013ed3d1f0
Assignee | ||
Comment 15•8 years ago
|
||
Whoa. It's bizarre that I never landed all of this. remote: https://hg.mozilla.org/build/tools/rev/51042994ea9c remote: https://hg.mozilla.org/build/tools/rev/c4b2c07f4d7a
Assignee | ||
Updated•8 years ago
|
Attachment #8583231 -
Flags: checkin+
Assignee | ||
Updated•8 years ago
|
Attachment #8583248 -
Flags: checkin+
Assignee | ||
Comment 17•8 years ago
|
||
Bleh. platform is "linux-debug". releng.manifest is under "linux32".
Attachment #8728550 -
Flags: review?(terrence)
Updated•8 years ago
|
Attachment #8728550 -
Flags: review?(terrence) → review+
Assignee | ||
Comment 18•8 years ago
|
||
Comment on attachment 8728550 [details] [diff] [review] use the correct name for linux32 releng.manifest https://hg.mozilla.org/build/tools/rev/f46262d5ecfc
Attachment #8728550 -
Flags: checkin+
Assignee | ||
Comment 19•8 years ago
|
||
Looks like this is all working, though I'm not sure why it only covers the simulator. Well, we'll see what bhearsum says in bug 1275775.
Status: ASSIGNED → RESOLVED
Closed: 8 years ago
Resolution: --- → FIXED
Comment 20•6 years ago
|
||
Removing leave-open keyword from resolved bugs, per :sylvestre.
Keywords: leave-open
You need to log in
before you can comment on or make changes to this bug.
Description
•