itsscb/cargo
1
0
Fork 0
mirror of https://github.com/rust-lang/cargo.git synced 2025-10-01 11:30:39 +00:00
Code Issues Packages Projects Releases Wiki Activity

fix(fingerprint): explicit reason than "stale; unknown reason"

Browse Source 
This was discovered during playing with rmeta reuse
between cargo-check and cargo-build
This commit is contained in:
Weihang Lo 2025-05-31 18:49:44 -04:00
parent 64a1246070
commit c6f5f3fcae
No known key found for this signature in database
GPG Key ID: D7DBF189825E82E7
1 changed files with 4 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

11
src/cargo/core/compiler/fingerprint/mod.rs
View File

@ -1167,15 +1167,12 @@ impl Fingerprint {
// minimum mtime as it's the one we'll be comparing to inputs and // minimum mtime as it's the one we'll be comparing to inputs and
// dependencies. // dependencies.
for output in self.outputs.iter() { for output in self.outputs.iter() {
let mtime = match paths::mtime(output) { let Ok(mtime) = paths::mtime(output) else {
Ok(mtime) => mtime,
// This path failed to report its `mtime`. It probably doesn't // This path failed to report its `mtime`. It probably doesn't
// exists, so leave ourselves as stale and bail out. // exists, so leave ourselves as stale and bail out.
Err(e) => { let item = StaleItem::FailedToReadMetadata(output.clone());
debug!("failed to get mtime of {:?}: {}", output, e); self.fs_status = FsStatus::StaleItem(item);
return Ok(()); return Ok(());
}
}; };
assert!(mtimes.insert(output.clone(), mtime).is_none()); assert!(mtimes.insert(output.clone(), mtime).is_none());
} }