From 87824bca6ba579498e98a4ccd447597fc01e0abf Mon Sep 17 00:00:00 2001 From: John Ericson Date: Wed, 12 Mar 2025 18:08:58 -0400 Subject: [PATCH] Avoid pointless mutation The code that was in between is now gone. We can just set `st` correctly the first time. --- src/libstore/unix/build/local-derivation-goal.cc | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/libstore/unix/build/local-derivation-goal.cc b/src/libstore/unix/build/local-derivation-goal.cc index f17b0bced..b2e4c1186 100644 --- a/src/libstore/unix/build/local-derivation-goal.cc +++ b/src/libstore/unix/build/local-derivation-goal.cc @@ -372,10 +372,8 @@ Goal::Co LocalDerivationGoal::tryLocalBuild() } catch (BuildError & e) { outputLocks.unlock(); - BuildResult::Status st = BuildResult::MiscFailure; - assert(derivationType); - st = + BuildResult::Status st = dynamic_cast(&e) ? BuildResult::NotDeterministic : statusOk(status) ? BuildResult::OutputRejected : !derivationType->isSandboxed() || diskFull ? BuildResult::TransientFailure :