Merge pull request #4326 from tweag/fix-post-build-hook-and-remote-builders

Re-query for the derivation outputs in the post-build-hook

1 files changed, 4 insertions, 2 deletions

diff --git a/src/libstore/build/derivation-goal.cc b/src/libstore/build/derivation-goal.cc
index 1db85bd37..fdf777c27 100644
--- a/src/libstore/build/derivation-goal.cc
+++ b/src/libstore/build/derivation-goal.cc
@@ -899,8 +899,10 @@ void DerivationGoal::buildDone()
                 Logger::Fields{worker.store.printStorePath(drvPath)});
             PushActivity pact(act.id);
             StorePathSet outputPaths;
-            for (auto i : drv->outputs) {
-                outputPaths.insert(finalOutputs.at(i.first));
+            for (auto& [_, maybeOutPath] :
+                 worker.store.queryPartialDerivationOutputMap(drvPath)) {
+                if (maybeOutPath)
+                    outputPaths.insert(*maybeOutPath);
             }
             std::map<std::string, std::string> hookEnvironment = getEnv();