From feeabe97af3252f6730ed63448b28b2d43a28544 Mon Sep 17 00:00:00 2001 From: atlas Date: Tue, 11 Aug 2026 22:29:56 +0200 Subject: [PATCH] docs(rust): the dep-cache comment no longer describes two consumers MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Follow-up to my own #3171. That PR gave the deploy path its own cache and left the paragraph above it saying "package builds and checks reuse this" — which stopped being true in the same commit that made it false. Merged into one accurate lead-in. The surviving facts (built once, every consumer passes the same `cleanSrc`) are still worth stating; the claim about who consumes it belongs to the two blocks below, which say it correctly. --- nix/rust.nix | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/nix/rust.nix b/nix/rust.nix index 99ab07a5..0ea9a604 100644 --- a/nix/rust.nix +++ b/nix/rust.nix @@ -40,11 +40,11 @@ rec { fileset = craneLib.fileset.commonCargoSources ../.; }; - # Build the workspace's dependency tree once, cached as its own - # derivation. Package builds and checks reuse this via - # `inherit cargoArtifacts;` so a workspace-only edit doesn't rebuild - # deps. All consumers use the same `cleanSrc` so the input hash - # stays consistent across the chain. + # The workspace's dependency tree, built once and cached as its own + # derivation so a workspace-only edit doesn't rebuild deps. Every + # consumer passes the same `cleanSrc`, keeping the input hash + # consistent across the chain. + # # Two dep caches, one per audience, and the split is not premature: # crane's `buildDepsOnly` defaults `doCheck = true`, which adds # `--all-targets` to the check and a `cargo test --no-run`, so the cache