aboutsummaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorThéophane Hufschmitt <7226587+thufschmitt@users.noreply.github.com>2022-12-05 18:04:41 +0100
committerGitHub <noreply@github.com>2022-12-05 18:04:41 +0100
commitbfcf30f0abdfd1df4364919f203d04ff08d7314c (patch)
tree817212321f06e8cf7af8b47629e5902f61cf234f
parenta21d9523b95fd7292a5a8825b1e772f7eda36ed9 (diff)
parente4f9f3bf246d66c57b07b45583469b98ba0db367 (diff)
Merge pull request #7390 from Radvendii/fix-6383
check the store for input before failing (hopefully fix #6700)
-rw-r--r--src/libstore/build/derivation-goal.cc8
1 files changed, 8 insertions, 0 deletions
diff --git a/src/libstore/build/derivation-goal.cc b/src/libstore/build/derivation-goal.cc
index 67cfc38af..5aed51bcd 100644
--- a/src/libstore/build/derivation-goal.cc
+++ b/src/libstore/build/derivation-goal.cc
@@ -501,6 +501,14 @@ void DerivationGoal::inputsRealised()
now-known results of dependencies. If so, we become a
stub goal aliasing that resolved derivation goal. */
std::optional attempt = fullDrv.tryResolve(worker.store, inputDrvOutputs);
+ if (!attempt) {
+ /* TODO (impure derivations-induced tech debt) (see below):
+ The above attempt should have found it, but because we manage
+ inputDrvOutputs statefully, sometimes it gets out of sync with
+ the real source of truth (store). So we query the store
+ directly if there's a problem. */
+ attempt = fullDrv.tryResolve(worker.store);
+ }
assert(attempt);
Derivation drvResolved { *std::move(attempt) };