Closed
Bug 1000723
Opened 11 years ago
Closed 11 years ago
Keep track of stdout/stderr in sccache
Categories
(Firefox Build System :: General, defect)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla32
People
(Reporter: glandium, Assigned: glandium)
References
(Blocks 1 open bug)
Details
Attachments
(3 files)
3.91 KB,
patch
|
mshal
:
review+
|
Details | Diff | Splinter Review |
1.20 KB,
patch
|
mshal
:
review+
|
Details | Diff | Splinter Review |
6.55 KB,
patch
|
Details | Diff | Splinter Review |
Currently, sccache doesn't store anything else than the object files in cache, so when there is a cache hit, we don't get anything that would normally be output without the cache, like warnings.
Assignee | ||
Comment 1•11 years ago
|
||
Assignee: nobody → mh+mozilla
Attachment #8411590 -
Flags: review?(mshal)
Assignee | ||
Comment 2•11 years ago
|
||
Attachment #8411592 -
Flags: review?(mshal)
Assignee | ||
Comment 3•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Updated•11 years ago
|
Attachment #8411592 -
Flags: review?(mshal) → review+
Comment 4•11 years ago
|
||
Comment on attachment 8411590 [details] [diff] [review]
Allow to store more than one object file in CacheData, and add stderr
Can you add some top-level documentation to CacheData? My understanding from the code is that each cache object is now a zip file that contains three "files", named "obj", "stdout", and "stderr", but it's not so obvious without looking at server.py.
Also some detail about when it is necessary to update CacheData.VERSION and when it's not would be helpful.
Attachment #8411590 -
Flags: review?(mshal) → review+
Assignee | ||
Comment 5•11 years ago
|
||
Assignee | ||
Updated•11 years ago
|
Status: NEW → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Target Milestone: --- → mozilla32
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
•