Closed Bug 1341962 Opened 3 years ago Closed 3 years ago

ccache stats parser breaks with ccache 3.3.3

Categories

(Firefox Build System :: General, defect)

defect
Not set

Tracking

(firefox54 fixed)

RESOLVED FIXED
mozilla54
Tracking Status
firefox54 --- fixed

People

(Reporter: TYLin, Assigned: TYLin)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

+++ This bug was initially created as a clone of Bug #1309657 +++

I upgrade to ccache 3.3.3 on Mac, and have the following error.

5:03.35 Failed to parse ccache stats output: unsupported code directive             2
Comment on attachment 8840276 [details]
Bug 1341962 - Update ccache stats parser for version 3.3.3.

https://reviewboard.mozilla.org/r/114766/#review117642
Attachment #8840276 - Flags: review?(mh+mozilla) → review+
Pushed by tlin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/2a15e34e2d22
Update ccache stats parser for version 3.3.3. r=glandium
I rebase my patch, and delete the stats_diff test because this patch just updated the parser to recognize a new field, which is not relevant for us for displaying the ccache status. A parsing test to the new format should be sufficient.
Flags: needinfo?(tlin)
Pushed by tlin@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/786dcb25ee48
Update ccache stats parser for version 3.3.3. r=glandium
https://hg.mozilla.org/mozilla-central/rev/786dcb25ee48
Status: NEW → RESOLVED
Closed: 3 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.