From 81401a17e5fe5e239ca99b48db14a67714cd9ae7 Mon Sep 17 00:00:00 2001 From: Charlie Marsh Date: Mon, 22 Jan 2024 23:51:18 -0500 Subject: [PATCH] Use `archive_mtime` in another call site (#1056) _Not_ using this was an oversight. --- crates/puffin-distribution/src/source/mod.rs | 26 ++------------------ 1 file changed, 2 insertions(+), 24 deletions(-) diff --git a/crates/puffin-distribution/src/source/mod.rs b/crates/puffin-distribution/src/source/mod.rs index 15404c7d8..a7931cbb7 100644 --- a/crates/puffin-distribution/src/source/mod.rs +++ b/crates/puffin-distribution/src/source/mod.rs @@ -537,30 +537,8 @@ impl<'a, T: BuildContext> SourceDistCachedBuilder<'a, T> { ); // Determine the last-modified time of the source distribution. - let file_metadata = fs_err::metadata(&path_source_dist.path)?; - let modified = if file_metadata.is_file() { - // `modified()` is infallible on windows and unix (i.e., all platforms we support). - file_metadata.modified()? - } else { - if let Some(metadata) = path_source_dist - .path - .join("pyproject.toml") - .metadata() - .ok() - .filter(std::fs::Metadata::is_file) - { - metadata.modified()? - } else if let Some(metadata) = path_source_dist - .path - .join("setup.py") - .metadata() - .ok() - .filter(std::fs::Metadata::is_file) - { - metadata.modified()? - } else { - return Err(SourceDistError::DirWithoutEntrypoint); - } + let Some(modified) = puffin_cache::archive_mtime(&path_source_dist.path)? else { + return Err(SourceDistError::DirWithoutEntrypoint); }; // Read the existing metadata from the cache.