From 51dadaded444907ecb97e19a34483f06d10d1ab5 Mon Sep 17 00:00:00 2001 From: John Ericson Date: Fri, 29 Aug 2025 14:40:41 -0400 Subject: [PATCH] Move up `assert(!hook);` We don't need to keep doing this every loop iteration, hook stuff it is only set above. --- src/libstore/build/derivation-building-goal.cc | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/src/libstore/build/derivation-building-goal.cc b/src/libstore/build/derivation-building-goal.cc index fd85a066d..510304653 100644 --- a/src/libstore/build/derivation-building-goal.cc +++ b/src/libstore/build/derivation-building-goal.cc @@ -602,12 +602,11 @@ Goal::Co DerivationBuildingGoal::tryToBuild() #ifdef _WIN32 // TODO enable `DerivationBuilder` on Windows throw UnimplementedError("building derivations is not yet implemented on Windows"); #else + assert(!hook); // Will continue here while waiting for a build user below while (true) { - assert(!hook); - unsigned int curBuilds = worker.getNrLocalBuilds(); if (curBuilds >= settings.maxBuildJobs) { outputLocks.unlock();