Open
Bug 2017746
Opened 23 days ago
Updated 6 hours ago
`./mach configure` always re-runs `./mach taskgraph` via `toolchain_task_definitions()` even if nothing has changed on disk
Categories
(Firefox Build System :: General, defect, P2)
Firefox Build System
General
Tracking
(Not tracked)
NEW
People
(Reporter: ahochheiden, Assigned: ahochheiden)
Details
Attachments
(2 files)
On Windows, this is ~8-9 seconds on subsequent ./mach configure runs, taking up a substantial amount of the overall runtime. We should be able to cache this result with minimal effort by just hashing the contents of taskcluster/ and only re-running when there are changes.
| Assignee | ||
Comment 1•23 days ago
|
||
| Assignee | ||
Comment 2•23 days ago
|
||
Updated•22 days ago
|
Attachment #9546276 -
Attachment description: Bug 2017746 - Cache `toolchain_task_definitions()` with `mach_func_cache` to speed up subsequent configure runs when there are no `taskcluster` changes r?#build → Bug 2017746 - Cache `toolchain_task_definitions()` with `mach_func_cache` to speed up subsequent configure runs when there are no `taskcluster` or `toolchain-resources` changes r?#build
Comment 3•8 days ago
|
||
The severity field is not set for this bug.
:ahochheiden, could you have a look please?
For more information, please visit BugBot documentation.
Flags: needinfo?(ahochheiden)
| Assignee | ||
Updated•8 days ago
|
Severity: -- → S4
Flags: needinfo?(ahochheiden)
Priority: -- → P2
Pushed by ahochheiden@mozilla.com:
https://github.com/mozilla-firefox/firefox/commit/a78d7bbbaf81
https://hg.mozilla.org/integration/autoland/rev/16d1d9e52e52
Add `mach_func_cache` decorator for persistent function result caching r=firefox-build-system-reviewers,sergesanspaille,glandium
https://github.com/mozilla-firefox/firefox/commit/6be7acf6dc97
https://hg.mozilla.org/integration/autoland/rev/fcb933d373e2
Cache `toolchain_task_definitions()` with `mach_func_cache` to speed up subsequent configure runs when there are no `taskcluster` or `toolchain-resources` changes r=firefox-build-system-reviewers,glandium
You need to log in
before you can comment on or make changes to this bug.
Description
•