Closed
Bug 1228703
Opened 9 years ago
Closed 9 years ago
Enable rust for MacOS X nightly builds
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(firefox45 fixed)
RESOLVED
FIXED
mozilla45
Tracking | Status | |
---|---|---|
firefox45 | --- | fixed |
People
(Reporter: rillian, Assigned: rillian)
References
Details
Attachments
(3 files, 3 obsolete files)
(deleted),
patch
|
mshal
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
mshal
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
mshal
:
review+
|
Details | Diff | Splinter Review |
Turns out bug 1183850 only enabled rust for mac debug builds, so the rust code isn't getting active testing in the Nightly population. We should turn it on there too.
Assignee | ||
Comment 1•9 years ago
|
||
Initial patch.
The mac opt builds use the macosx-universal mozconfigs, building both 32 and 64 bit versions. This patch fails in the 32 bit configure when the rust link check fails to find 32 bit symbols. So we either need to pass --target=i686-apple-darwin to rustc in this case, or make the mozconfig.rust inclusion conditional to exclude 32-bit.
Either is fine in the short term, the building for both archs is the correct long-term fix.
Comment 2•9 years ago
|
||
I say we go the 64-bit only route, as we'd like to make 32-bit OS X builds go away in the not-too-distant future.
Assignee | ||
Comment 3•9 years ago
|
||
Ok, great. Do you know what I can check for the target in the mozconfig?
Assignee | ||
Comment 4•9 years ago
|
||
Try setting the cross triple for the universal build in client.mk based on MOZ_CURRENT_TARGET. We don't get to find out if this works because we still fail on configure's link test which happens before this.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=72b2a84ec531&selectedJob=14282124
Assignee | ||
Comment 5•9 years ago
|
||
Next idea: check for HAVE_64BIT_BUILD in configure and append the 32-bit triple to RUSTC if it's not set and we're building on Mac.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=c660a4ac69a9
Attachment #8695373 -
Attachment is obsolete: true
Assignee | ||
Comment 6•9 years ago
|
||
Attachment #8696171 -
Flags: review?(mshal)
Assignee | ||
Comment 7•9 years ago
|
||
Set cross triple in configure based on HAVE_64BIT_BUILD. This method works.
https://treeherder.mozilla.org/#/jobs?repo=try&revision=557a5d3ca194&selectedJob=14432388
Attachment #8695383 -
Attachment is obsolete: true
Attachment #8696174 -
Flags: review?(mshal)
Assignee | ||
Comment 8•9 years ago
|
||
Attachment #8693143 -
Attachment is obsolete: true
Attachment #8696175 -
Flags: review?(mshal)
Assignee | ||
Comment 9•9 years ago
|
||
Full try push looks ok. https://treeherder.mozilla.org/#/jobs?repo=try&revision=f45c20e27f4f&selectedJob=14441544
Updated•9 years ago
|
Attachment #8696171 -
Flags: review?(mshal) → review+
Updated•9 years ago
|
Attachment #8696174 -
Flags: review?(mshal) → review+
Comment 10•9 years ago
|
||
Comment on attachment 8696175 [details] [diff] [review]
Part 3 - Enable rust for MacOS X nightly builds
>Subject: Bug 1228703 - Enable rust for MacOS X nightly builds. r?mhal
mhal -> mshal :)
Everything looks good to me.
Attachment #8696175 -
Flags: review?(mshal) → review+
Assignee | ||
Comment 11•9 years ago
|
||
(In reply to Michael Shal [:mshal] from comment #10)
> mhal -> mshal :)
Oops, sorry! Thanks for the meeting-time review. :)
Comment 12•9 years ago
|
||
Comment 13•9 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/a0920d7f00a0
https://hg.mozilla.org/mozilla-central/rev/90136d8f3801
https://hg.mozilla.org/mozilla-central/rev/bc8ecea2a286
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla45
Updated•7 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•