Don't cache realiseContext() errors

Errors that depend on the configuration (such as whether allow-import-from-derivation is set) should not be cached.
klemensn · Sep 22, 2021 · ff28fff · ff28fff
1 parent bcd73eb
commit ff28fff
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/src/libexpr/primops.cc b/src/libexpr/primops.cc
@@ -52,7 +52,8 @@ void EvalState::realiseContext(const PathSet & context)
     if (drvs.empty()) return;
 
     if (!evalSettings.enableImportFromDerivation)
-        throw EvalError("attempted to realize '%1%' during evaluation but 'allow-import-from-derivation' is false",
+        throw Error(
+            "cannot build '%1%' during evaluation because the option 'allow-import-from-derivation' is disabled",
             store->printStorePath(drvs.begin()->drvPath));
 
     /* For performance, prefetch all substitute info. */