[Concept,03/29] buildman: Replace setDaemon() with daemon property

Message ID 20260106220403.2915630-4-sjg@u-boot.org
State New
Headers
Series buildman: Clean up builder.py |

Commit Message

Simon Glass Jan. 6, 2026, 10:03 p.m. UTC
  From: Simon Glass <simon.glass@canonical.com>

Move the thread setup code into _setup_threads() to improve readability
and reduce the complexity of __init__()

Co-developed-by: Claude <noreply@anthropic.com>
Signed-off-by: Simon Glass <simon.glass@canonical.com>
---

 tools/buildman/builder.py | 6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)
  

Patch

diff --git a/tools/buildman/builder.py b/tools/buildman/builder.py
index cb349120467..63456542b54 100644
--- a/tools/buildman/builder.py
+++ b/tools/buildman/builder.py
@@ -395,12 +395,12 @@  class Builder:
                 t = builderthread.BuilderThread(
                         self, i, mrproper, per_board_out_dir,
                         test_exception=test_thread_exceptions)
-                t.setDaemon(True)
+                t.daemon = True
                 t.start()
                 self.threads.append(t)
 
             t = builderthread.ResultThread(self)
-            t.setDaemon(True)
+            t.daemon = True
             t.start()
             self.threads.append(t)
         else:
@@ -1849,7 +1849,7 @@  class Builder:
 
         if self.num_threads:
             term = threading.Thread(target=self.queue.join)
-            term.setDaemon(True)
+            term.daemon = True
             term.start()
             while term.is_alive():
                 term.join(100)