From 5f6ff55e97104e46c1b5c2c94b39ea0fca35ca7c Mon Sep 17 00:00:00 2001
From: Paul Cristian Sarbu <paul.cristian.sarbu@huawei.com>
Date: Thu, 7 Mar 2024 15:44:28 +0100
Subject: just: Replace singletons for progress tracking and statistics...

...with regular instances that have controlled life-times.

This avoids race conditions in tracking and reporting the results
of analysis and build, as the serve endpoint can orchestrate
multiple builds at the same time asynchronously. As a bonus
side-effect this also ensures the correctness of the progress
reporting per orchestrated build.
---
 src/buildtool/main/analyse.hpp | 2 ++
 1 file changed, 2 insertions(+)

(limited to 'src/buildtool/main/analyse.hpp')

diff --git a/src/buildtool/main/analyse.hpp b/src/buildtool/main/analyse.hpp
index a7e1cb08..c3b4917a 100644
--- a/src/buildtool/main/analyse.hpp
+++ b/src/buildtool/main/analyse.hpp
@@ -20,6 +20,7 @@
 #include "src/buildtool/build_engine/target_map/result_map.hpp"
 #include "src/buildtool/common/cli.hpp"
 #include "src/buildtool/common/repository_config.hpp"
+#include "src/buildtool/common/statistics.hpp"
 #include "src/buildtool/storage/target_cache.hpp"
 
 struct AnalysisResult {
@@ -33,6 +34,7 @@ struct AnalysisResult {
     gsl::not_null<BuildMaps::Target::ResultTargetMap*> const& result_map,
     gsl::not_null<RepositoryConfig*> const& repo_config,
     ActiveTargetCache const& target_cache,
+    gsl::not_null<Statistics*> const& stats,
     std::size_t jobs,
     std::optional<std::string> const& request_action_input)
     -> std::optional<AnalysisResult>;
-- 
cgit v1.2.3