[7/9] fix: handle directories in dep-info source file hashing#5
Open
mmastrac wants to merge 1 commit intovercel/file-clone-compressfrom
Open
[7/9] fix: handle directories in dep-info source file hashing#5mmastrac wants to merge 1 commit intovercel/file-clone-compressfrom
mmastrac wants to merge 1 commit intovercel/file-clone-compressfrom
Conversation
5344dad to
f487cc2
Compare
9ea5c59 to
59848e7
Compare
f487cc2 to
a7bc6d3
Compare
79c28ed to
afd0422
Compare
b668492 to
c779b68
Compare
19a11db to
684ec28
Compare
226e9df to
c3ebfff
Compare
3f6e50b to
7ad84a3
Compare
2afce2b to
f12d053
Compare
7ad84a3 to
e491a69
Compare
f12d053 to
136498c
Compare
e491a69 to
0237f6e
Compare
136498c to
4be87c8
Compare
9cefa6c to
a321d4a
Compare
4be87c8 to
0106a7d
Compare
a321d4a to
bf0aca6
Compare
ddb7117 to
eed7dde
Compare
bf0aca6 to
9590506
Compare
9590506 to
bd2a312
Compare
eed7dde to
f7e3b82
Compare
bd2a312 to
62e8073
Compare
3d27feb to
cf104ad
Compare
62e8073 to
03f037d
Compare
cf104ad to
66992c5
Compare
03f037d to
3e055b2
Compare
66992c5 to
dd8329f
Compare
3e055b2 to
37ccf40
Compare
37ccf40 to
db0b087
Compare
dd8329f to
0511c15
Compare
db0b087 to
4fdbbf4
Compare
fbb6657 to
5e9fb55
Compare
4fdbbf4 to
2715294
Compare
When proc_macro::tracked::path() registers a directory as a dependency, rustc's dep-info output includes the directory path. sccache previously crashed with "Is a directory" when trying to hash these paths. Now recursively hashes all files within the directory (sorted for determinism), using relative paths as delimiters. This correctly captures directory dependencies so cache invalidation works when any file in the tracked directory changes. Fixes: mozilla#2653 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
2715294 to
d59ecfb
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Fixes mozilla#2653.
proc_macro::tracked::path()registers a directory as a dependency, rustc's dep-info includes the directory pathturbopack-nodethat useinclude_dir!with nightly proc macro trackingStack: 5/6