From 6378bad3853d632530ff7b17a2a6b8ce6360a843 Mon Sep 17 00:00:00 2001 From: Simon Glass Date: Wed, 19 Jul 2023 17:48:50 -0600 Subject: buildman: Move commit numbering into determine_series() Commits are numbered for use in tests. Do this in determine_series() since it is already dealing with the series. Signed-off-by: Simon Glass --- tools/buildman/control.py | 10 +++++++--- 1 file changed, 7 insertions(+), 3 deletions(-) (limited to 'tools') diff --git a/tools/buildman/control.py b/tools/buildman/control.py index 26f0db47684..51dc6fe7ce7 100644 --- a/tools/buildman/control.py +++ b/tools/buildman/control.py @@ -202,6 +202,9 @@ def count_commits(branch, count, col, git_dir): def determine_series(selected, col, git_dir, count, branch, work_in_output): """Determine the series which is to be built, if any + If there is a series, the commits in that series are numbered by setting + their sequence value (starting from 0). This is used by tests. + Args: selected (list of Board): List of Board objects that are marked selected @@ -254,6 +257,10 @@ def determine_series(selected, col, git_dir, count, branch, work_in_output): # Honour the count series = patchstream.get_metadata_for_list(branch, git_dir, count, series=None, allow_overwrite=True) + + # Number the commits for test purposes + for i, commit in enumerate(series.commits): + commit.sequence = i else: series = None return series @@ -580,9 +587,6 @@ def do_buildman(options, args, toolchains=None, make_func=None, brds=None, if series: commits = series.commits - # Number the commits for test purposes - for i, commit in enumerate(commits): - commit.sequence = i else: commits = None -- cgit v1.2.3