diff --git a/src/libstore/build/derivation-goal.cc b/src/libstore/build/derivation-goal.cc index c288003b4..3011d5c6f 100644 --- a/src/libstore/build/derivation-goal.cc +++ b/src/libstore/build/derivation-goal.cc @@ -1549,7 +1549,7 @@ Goal::Finished DerivationGoal::done( } return Finished{ - .result = buildResult.success() ? ecSuccess : ecFailed, + .exitCode = buildResult.success() ? ecSuccess : ecFailed, .ex = ex ? std::make_shared(std::move(*ex)) : nullptr, .permanentFailure = buildResult.status == BuildResult::PermanentFailure, .timedOut = buildResult.status == BuildResult::TimedOut, diff --git a/src/libstore/build/goal.hh b/src/libstore/build/goal.hh index 1f25fb233..9d9ab53ca 100644 --- a/src/libstore/build/goal.hh +++ b/src/libstore/build/goal.hh @@ -124,7 +124,7 @@ public: bool inBuildSlot; }; struct [[nodiscard]] Finished { - ExitCode result; + ExitCode exitCode; std::shared_ptr ex; bool permanentFailure = false; bool timedOut = false; diff --git a/src/libstore/build/worker.cc b/src/libstore/build/worker.cc index 0642754e8..f619d574d 100644 --- a/src/libstore/build/worker.cc +++ b/src/libstore/build/worker.cc @@ -156,7 +156,7 @@ void Worker::goalFinished(GoalPtr goal, Goal::Finished & f) { goal->trace("done"); assert(!goal->exitCode.has_value()); - goal->exitCode = f.result; + goal->exitCode = f.exitCode; goal->ex = f.ex; permanentFailure |= f.permanentFailure; @@ -171,11 +171,11 @@ void Worker::goalFinished(GoalPtr goal, Goal::Finished & f) waiting->trace(fmt("waitee '%s' done; %d left", goal->name, waiting->waitees.size())); - if (f.result != Goal::ecSuccess) ++waiting->nrFailed; - if (f.result == Goal::ecNoSubstituters) ++waiting->nrNoSubstituters; - if (f.result == Goal::ecIncompleteClosure) ++waiting->nrIncompleteClosure; + if (f.exitCode != Goal::ecSuccess) ++waiting->nrFailed; + if (f.exitCode == Goal::ecNoSubstituters) ++waiting->nrNoSubstituters; + if (f.exitCode == Goal::ecIncompleteClosure) ++waiting->nrIncompleteClosure; - if (waiting->waitees.empty() || (f.result == Goal::ecFailed && !settings.keepGoing)) { + if (waiting->waitees.empty() || (f.exitCode == Goal::ecFailed && !settings.keepGoing)) { /* If we failed and keepGoing is not set, we remove all remaining waitees. */ for (auto & i : waiting->waitees) {