Closed
Bug 1430087
Opened 6 years ago
Closed 6 years ago
Build sccache with system GCC/binutils
Categories
(Firefox Build System :: General, enhancement)
Firefox Build System
General
Tracking
(firefox59 fixed)
RESOLVED
FIXED
mozilla59
Tracking | Status | |
---|---|---|
firefox59 | --- | fixed |
People
(Reporter: glandium, Assigned: glandium)
References
Details
Attachments
(1 file)
No description provided.
Comment hidden (mozreview-request) |
Comment 2•6 years ago
|
||
mozreview-review |
Comment on attachment 8942160 [details] Bug 1430087 - Build sccache with system GCC/binutils. https://reviewboard.mozilla.org/r/212430/#review218172 Does this mean we're OK with building the openssl required here with whatever random `cc` is available?
Attachment #8942160 -
Flags: review+
Comment 3•6 years ago
|
||
> This makes it now possible to use sccache as a dependency for all other
toolchains jobs that compile, if that's beneficial (which might not be
the case, given the current sccache retention time, but at least it's a
viable option, now)
I know we discussed this at some point but I can't remember what we decided. Using sccache for other toolchain jobs should at least help make the "dozens of toolchain jobs pile up when a toolchain change lands" situation better, since tasks that are started while a task is still running on a previous push ought to be able to get cache hits. We could also create a separate s3 bucket for toolchain use with a longer expiration date, or add s3 prefix support to sccache and put a prefix in the existing sccache buckets for toolchain use with a longer expiration date.
Comment 4•6 years ago
|
||
Comment on attachment 8942160 [details] Bug 1430087 - Build sccache with system GCC/binutils. See comment 2.
Attachment #8942160 -
Flags: review?(core-build-config-reviews) → review+
Assignee | ||
Comment 5•6 years ago
|
||
(In reply to Nathan Froyd [:froydnj] from comment #2) > Comment on attachment 8942160 [details] > Bug 1430087 - Build sccache with system GCC/binutils. > > https://reviewboard.mozilla.org/r/212430/#review218172 > > Does this mean we're OK with building the openssl required here with > whatever random `cc` is available? Why shouldn't we (it's GCC 4.7, fwiw)
Pushed by mh@glandium.org: https://hg.mozilla.org/integration/mozilla-inbound/rev/fd14af0fbb8f Build sccache with system GCC/binutils. r=nfroyd
Comment 7•6 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/fd14af0fbb8f
Status: NEW → RESOLVED
Closed: 6 years ago
status-firefox59:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla59
Updated•6 years ago
|
Product: Core → Firefox Build System
You need to log in
before you can comment on or make changes to this bug.
Description
•