[Concept,06/20] buildman: Initialise _timestamps to an empty deque

Message ID 20260316154733.1587261-7-sjg@u-boot.org
State New
Headers
Series buildman: Add distributed builds |

Commit Message

Simon Glass March 16, 2026, 3:47 p.m. UTC
  From: Simon Glass <sjg@chromium.org>

The demand-driven worker mode (introduced later in this series)
queues boards one at a time and may call process_result() before
the first board has been set up, which crashes on None.append()

Change _timestamps from None to an empty deque. The existing
build_boards() flow always calls _setup_build() before process_result(),
so this is safe for current callers.

Signed-off-by: Simon Glass <sjg@chromium.org>
---

 tools/buildman/builder.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)
  

Patch

diff --git a/tools/buildman/builder.py b/tools/buildman/builder.py
index 2c8d5c5b17b..f704cc86943 100644
--- a/tools/buildman/builder.py
+++ b/tools/buildman/builder.py
@@ -362,7 +362,7 @@  class Builder:
         self.commit_count = 0
         self.commits = None
         self.count = 0
-        self._timestamps = None
+        self._timestamps = collections.deque()
         self._verbose = False
 
         # Note: baseline state for result summaries is now in ResultHandler