{Local,Bazel}Response: Fix logic for marking succesful population

While in practice a failure to populate the fields of a response
happens once per invocation, as it will trigger a failure of the
execution, from an algorithmic standpoint the flag to mark a
successful population of the response fields should only be set on
actual success.

Fix this.

1 files changed, 1 insertions, 1 deletions

diff --git a/src/buildtool/execution_api/local/local_response.hpp b/src/buildtool/execution_api/local/local_response.hpp
index 852284b7..790bb384 100644
--- a/src/buildtool/execution_api/local/local_response.hpp
+++ b/src/buildtool/execution_api/local/local_response.hpp
@@ -144,7 +144,6 @@ class LocalResponse final : public IExecutionResponse {
         if (populated_) {
             return std::nullopt;
         }
-        populated_ = true;
 
         ArtifactInfos artifacts{};
         auto const& action_result = output_.action;
@@ -276,6 +275,7 @@ class LocalResponse final : public IExecutionResponse {
         }
         artifacts_ = std::move(artifacts);
         dir_symlinks_ = std::move(dir_symlinks);
+        populated_ = true;
         return std::nullopt;
     }