From 45b5c606ac44550de14562df4fa99773a81a1015 Mon Sep 17 00:00:00 2001
From: Eelco Dolstra <edolstra@gmail.com>
Date: Tue, 4 Jun 2019 20:34:08 +0200
Subject: [PATCH] Don't register invalid paths as GC roots

Unfortunately this doesn't work. Maybe we should keep separate roots
for each path.
---
 src/nix/installables.cc | 11 ++++++++---
 1 file changed, 8 insertions(+), 3 deletions(-)

diff --git a/src/nix/installables.cc b/src/nix/installables.cc
index e7549b57c..ca88ec0da 100644
--- a/src/nix/installables.cc
+++ b/src/nix/installables.cc
@@ -207,11 +207,16 @@ void makeFlakeClosureGCRoot(Store & store,
         /* Note: due to lazy fetching, these paths might not exist
            yet. */
         for (auto & dep : flake.flakeInputs) {
-            closure.insert(dep.second.computeStorePath(store));
+            auto path = dep.second.computeStorePath(store);
+            if (store.isValidPath(path))
+                closure.insert(path);
             queue.push(dep.second);
         }
-        for (auto & dep : flake.nonFlakeInputs)
-            closure.insert(dep.second.computeStorePath(store));
+        for (auto & dep : flake.nonFlakeInputs) {
+            auto path = dep.second.computeStorePath(store);
+            if (store.isValidPath(path))
+                closure.insert(path);
+        }
     }
 
     if (closure.empty()) return;