aboutsummaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorAlexander Bantyev <balsoft@balsoft.ru>2021-11-10 14:56:22 +0300
committerAlexander Bantyev <balsoft@balsoft.ru>2021-11-11 12:05:18 +0300
commit07bffe799853ef5e2757b9892e4e3dac89bfccbb (patch)
treedd018691cea9213888a7e45438f6c350bba95920 /src
parent52a3b2ee6367306a3f17fe9a2d0e2e4096852e05 (diff)
Flakes: refetch the input when a follows disappears
When an input follows disappears, we can't just reuse the old lock file entries since we may be missing some required ones. Refetch the input when this happens. Closes https://github.com/NixOS/nix/issues/5289
Diffstat (limited to 'src')
-rw-r--r--src/libexpr/flake/flake.cc14
1 files changed, 14 insertions, 0 deletions
diff --git a/src/libexpr/flake/flake.cc b/src/libexpr/flake/flake.cc
index 07ed3caa2..1dc6f5694 100644
--- a/src/libexpr/flake/flake.cc
+++ b/src/libexpr/flake/flake.cc
@@ -462,6 +462,8 @@ LockedFlake lockFlake(
those. */
FlakeInputs fakeInputs;
+ bool refetch = false;
+
for (auto & i : oldLock->inputs) {
if (auto lockedNode = std::get_if<0>(&i.second)) {
fakeInputs.emplace(i.first, FlakeInput {
@@ -469,12 +471,24 @@ LockedFlake lockFlake(
.isFlake = (*lockedNode)->isFlake,
});
} else if (auto follows = std::get_if<1>(&i.second)) {
+ auto o = input.overrides.find(i.first);
+ // If the override disappeared, we have to refetch the flake,
+ // since some of the inputs may not be present in the lockfile.
+ if (o == input.overrides.end()) {
+ refetch = true;
+ // There's no point populating the rest of the fake inputs,
+ // since we'll refetch the flake anyways.
+ break;
+ }
fakeInputs.emplace(i.first, FlakeInput {
.follows = *follows,
});
}
}
+ if (refetch)
+ fakeInputs = getFlake(state, oldLock->lockedRef, false, flakeCache).inputs;
+
computeLocks(fakeInputs, childNode, inputPath, oldLock, parent, parentPath);
}