Tup build fails with ccache enabled ("tup error: File '~/.ccache/7/stats' was written to, but is not in .tup/db")

RESOLVED FIXED in Firefox 63

Status

defect
RESOLVED FIXED
Last year
11 months ago

People

(Reporter: botond, Assigned: mshal)

Tracking

(Blocks 1 bug)

Trunk
mozilla63
Dependency tree / graph

Firefox Tracking Flags

(firefox63 fixed)

Details

Attachments

(2 attachments)

I just tried to build with Tup as suggested in mshal's recent post to dev-builds. The build failed with the following error:

 1:00.70  *** tup errors ***
 1:00.70 tup error: File '/home/botond/.ccache/7/stats' was written to, but is not in .tup/db. You probably should specify it as an output
 1:00.70  -- Delete: /home/botond/.ccache/7/stats
 1:00.70  *** Command failed due to errors processing the output dependencies.

I've attached the complete build output.
Summary: Tup build fails with "tup error: File '/home/botond/.ccache/7/stats' was written to, but is not in .tup/db" → Tup build fails with "tup error: File '~/.ccache/7/stats' was written to, but is not in .tup/db"
Ahh, I think we were so focused on eventually supporting sccache that I completely forgot about regular ccache. Thanks for filing this - I'll take a look.
Assignee: nobody → mshal
Summary: Tup build fails with "tup error: File '~/.ccache/7/stats' was written to, but is not in .tup/db" → Tup build fails with ccache enabled ("tup error: File '~/.ccache/7/stats' was written to, but is not in .tup/db")
So I was able to get this to work by ignoring file accesses to .ccache directories. One caveat here is I had to disable ccache's direct mode, otherwise it will hide some header dependencies from tup and potentially break the build. This means it always uses the preprocessor, which is slower than direct mode. However, I view the slight speed penalty worth not having a broken build.

Since this is a fix in tup itself, we just need to bump the revision in the toolchain. I didn't tag a release for this to avoid churn for packagers, since I expect to make a few more changes to tup in the near-term. This lets us use ccache in the meantime, though.
This includes a patch to enable ccache support.

MozReview-Commit-ID: 9FzMQ2XX4ca
Comment on attachment 9000069 [details]
Bug 1480840 - Update tup toolchain to f77dbd429d6157dac252014a26f6d275dce85d85; r?chmanchester

Chris Manchester (:chmanchester) has approved the revision.
Attachment #9000069 - Flags: review+
Pushed by mshal@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/279f7c58913c
Update tup toolchain to f77dbd429d6157dac252014a26f6d275dce85d85; r=chmanchester
https://hg.mozilla.org/mozilla-central/rev/279f7c58913c
Status: NEW → RESOLVED
Closed: 11 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla63
You need to log in before you can comment on or make changes to this bug.