Misc mac build config cleanups
Categories
(Firefox Build System :: General, task)
Tracking
(firefox84 fixed, firefox85 fixed)
People
(Reporter: glandium, Assigned: glandium)
References
Details
Attachments
(3 files)
47 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details | Review |
47 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details | Review |
47 bytes,
text/x-phabricator-request
|
RyanVM
:
approval-mozilla-beta+
|
Details | Review |
Assignee | ||
Updated•4 years ago
|
Assignee | ||
Comment 1•4 years ago
|
||
This hasn't been necessary since bug 1526857.
Assignee | ||
Comment 2•4 years ago
|
||
And don't set it via mozconfig. The default to /System/Library/PrivateFrameworks
may also not have matched the used SDK previously, so the new default it
better.
Assignee | ||
Comment 3•4 years ago
|
||
There is no earlier SDK that supports it. It seems Xcode clang doesn't
care (maybe it defaults to 11.0 is MACOSX_DEPLOYMENT_TARGET is too low?),
but upstream clang does.
Pushed by mh@glandium.org: https://hg.mozilla.org/integration/autoland/rev/b1e3d4009c17 Don't set BINDGEN_CFLAGS in mac mozconfig. r=firefox-build-system-reviewers,mhentges https://hg.mozilla.org/integration/autoland/rev/5588b2b9a15e Default to use private frameworks from the SDK. r=firefox-build-system-reviewers,mhentges https://hg.mozilla.org/integration/autoland/rev/d2312de47d56 Default to 11.0 as the macos target for aarch64. r=firefox-build-system-reviewers,mhentges
Comment 5•4 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/b1e3d4009c17
https://hg.mozilla.org/mozilla-central/rev/5588b2b9a15e
https://hg.mozilla.org/mozilla-central/rev/d2312de47d56
Assignee | ||
Comment 6•4 years ago
|
||
Comment on attachment 9188817 [details]
Bug 1678291 - Default to 11.0 as the macos target for aarch64.
Beta/Release Uplift Approval Request
- User impact if declined: Indirect dependency for bug 1678485
- Is this code covered by automated tests?: No
- Has the fix been verified in Nightly?: No
- Needs manual test from QE?: No
- If yes, steps to reproduce:
- List of other uplifts needed: None
- Risk to taking this patch: Low
- Why is the change risky/not risky? (and alternatives if risky): These patches are, in practice, no-ops for all builds, including Apple Silicon ones. They are only preparing things such that bug 1678154 doesn't break.
- String changes made/needed:
Assignee | ||
Updated•4 years ago
|
Comment 7•4 years ago
|
||
Comment on attachment 9188815 [details]
Bug 1678291 - Don't set BINDGEN_CFLAGS in mac mozconfig.
Approved for 84.0b4.
Updated•4 years ago
|
Updated•4 years ago
|
Comment 8•4 years ago
|
||
bugherder uplift |
https://hg.mozilla.org/releases/mozilla-beta/rev/2884e586f703
https://hg.mozilla.org/releases/mozilla-beta/rev/36c29c0432e8
https://hg.mozilla.org/releases/mozilla-beta/rev/c4f2375b30e3
Description
•