From 0b5c8cfda0f62c47a1f203dc2d182733e4196c34 Mon Sep 17 00:00:00 2001
From: Francois Kritzinger <francois@codesynthesis.com>
Date: Wed, 17 Apr 2024 08:15:54 +0200
Subject: Also pass build hints to build_building() and build_built()

---
 mod/mod-build-task.cxx | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

(limited to 'mod/mod-build-task.cxx')

diff --git a/mod/mod-build-task.cxx b/mod/mod-build-task.cxx
index 6be77f6..305aaca 100644
--- a/mod/mod-build-task.cxx
+++ b/mod/mod-build-task.cxx
@@ -1109,7 +1109,7 @@ handle (request& rq, response& rs)
       const tenant_service_build_queued* tsq (nullptr);
       optional<pair<tenant_service, shared_ptr<build>>> tss;
       vector<build> qbs;
-      tenant_service_build_queued::build_queued_hints qhs;
+      tenant_service_base::build_hints bhs;
       optional<build_state> initial_state;
       bool rebuild_forced_build (false);
       bool rebuild_interrupted_rebuild (false);
@@ -1181,14 +1181,14 @@ handle (request& rq, response& rs)
         return r;
       };
 
-      auto queue_hints = [this] (const build_package& p)
+      auto build_hints = [this] (const build_package& p)
       {
         buildable_package_count tpc (
           build_db_->query_value<buildable_package_count> (
             query<buildable_package_count>::build_tenant::id == p.id.tenant));
 
-        return tenant_service_build_queued::build_queued_hints {
-          tpc == 1, p.configs.size () == 1};
+        return tenant_service_base::build_hints {tpc == 1,
+                                                 p.configs.size () == 1};
       };
 
       // Collect the auxiliary machines required for testing of the specified
@@ -1975,7 +1975,7 @@ handle (request& rq, response& rs)
                           (*initial_state != build_state::queued &&
                            !rebuild_forced_build))
                       {
-                        qhs = queue_hints (*p);
+                        bhs = build_hints (*p);
 
                         t->queued_timestamp = system_clock::now ();
                         build_db_->update (t);
@@ -2215,6 +2215,8 @@ handle (request& rq, response& rs)
                       tsb = dynamic_cast<const tenant_service_build_building*> (s);
                       tsq = dynamic_cast<const tenant_service_build_queued*> (s);
 
+                      bhs = build_hints (*p);
+
                       if (tsq != nullptr)
                       {
                         qbs = queue_builds (*p, *b);
@@ -2228,8 +2230,6 @@ handle (request& rq, response& rs)
                         //
                         if (!qbs.empty () || !rebuild_interrupted_rebuild)
                         {
-                          qhs = queue_hints (*p);
-
                           t->queued_timestamp = system_clock::now ();
                           build_db_->update (t);
                         }
@@ -2312,7 +2312,7 @@ handle (request& rq, response& rs)
           if (auto f = tsq->build_queued (ss,
                                           qbs,
                                           nullopt /* initial_state */,
-                                          qhs,
+                                          bhs,
                                           log_writer_))
           {
             conn = build_db_->connection ();
@@ -2344,7 +2344,7 @@ handle (request& rq, response& rs)
           if (auto f = tsq->build_queued (ss,
                                           qbs,
                                           initial_state,
-                                          qhs,
+                                          bhs,
                                           log_writer_))
           {
             conn = build_db_->connection ();
-- 
cgit v1.1