From 82cae74799e5a64c819556f6152ba3734f1e2035 Mon Sep 17 00:00:00 2001 From: Paul Cristian Sarbu Date: Tue, 27 May 2025 14:51:06 +0200 Subject: {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. --- src/buildtool/execution_api/local/local_response.hpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'src/buildtool/execution_api/local/local_response.hpp') 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; } -- cgit v1.2.3