]> git.dujemihanovic.xyz Git - u-boot.git/commitdiff
buildman: Avoid using board as a variable
authorSimon Glass <sjg@chromium.org>
Tue, 12 Jul 2022 01:03:57 +0000 (19:03 -0600)
committerTom Rini <trini@konsulko.com>
Fri, 5 Aug 2022 15:47:55 +0000 (11:47 -0400)
We have a module called 'board'. Sometimes buildman uses 'brd' as an
instance variable but sometimes it uses 'board', which is confusing and
can mess with the module handling. Update the code to use 'brd'
consistently, making it easier for tools to determine when the module
is being referenced.

Signed-off-by: Simon Glass <sjg@chromium.org>
tools/buildman/board.py
tools/buildman/builder.py
tools/buildman/builderthread.py
tools/buildman/func_test.py
tools/buildman/test.py
tools/buildman/toolchain.py

index 447aaabea86a6be89444b7842b8ffd52d5c54705..08771b3f15828fefe7d1937ea884bd651dae2c7a 100644 (file)
@@ -103,15 +103,15 @@ class Boards:
         # Use a simple list here, sinc OrderedDict requires Python 2.7
         self._boards = []
 
-    def AddBoard(self, board):
+    def AddBoard(self, brd):
         """Add a new board to the list.
 
         The board's target member must not already exist in the board list.
 
         Args:
-            board: board to add
+            brd: board to add
         """
-        self._boards.append(board)
+        self._boards.append(brd)
 
     def ReadBoards(self, fname):
         """Read a list of boards from a board file.
@@ -136,8 +136,8 @@ class Boards:
                 if len(fields) > 8:
                     fields = fields[:8]
 
-                board = Board(*fields)
-                self.AddBoard(board)
+                brd = Board(*fields)
+                self.AddBoard(brd)
 
 
     def GetList(self):
@@ -157,8 +157,8 @@ class Boards:
                 value is board
         """
         board_dict = OrderedDict()
-        for board in self._boards:
-            board_dict[board.target] = board
+        for brd in self._boards:
+            board_dict[brd.target] = brd
         return board_dict
 
     def GetSelectedDict(self):
@@ -168,9 +168,9 @@ class Boards:
             List of Board objects that are marked selected
         """
         board_dict = OrderedDict()
-        for board in self._boards:
-            if board.build_it:
-                board_dict[board.target] = board
+        for brd in self._boards:
+            if brd.build_it:
+                board_dict[brd.target] = brd
         return board_dict
 
     def GetSelected(self):
@@ -179,7 +179,7 @@ class Boards:
         Returns:
             List of Board objects that are marked selected
         """
-        return [board for board in self._boards if board.build_it]
+        return [brd for brd in self._boards if brd.build_it]
 
     def GetSelectedNames(self):
         """Return a list of selected boards
@@ -187,7 +187,7 @@ class Boards:
         Returns:
             List of board names that are marked selected
         """
-        return [board.target for board in self._boards if board.build_it]
+        return [brd.target for brd in self._boards if brd.build_it]
 
     def _BuildTerms(self, args):
         """Convert command line arguments to a list of terms.
@@ -273,34 +273,34 @@ class Boards:
             exclude_list.append(Expr(expr))
 
         found = []
-        for board in self._boards:
+        for brd in self._boards:
             matching_term = None
             build_it = False
             if terms:
                 match = False
                 for term in terms:
-                    if term.Matches(board.props):
+                    if term.Matches(brd.props):
                         matching_term = str(term)
                         build_it = True
                         break
             elif boards:
-                if board.target in boards:
+                if brd.target in boards:
                     build_it = True
-                    found.append(board.target)
+                    found.append(brd.target)
             else:
                 build_it = True
 
             # Check that it is not specifically excluded
             for expr in exclude_list:
-                if expr.Matches(board.props):
+                if expr.Matches(brd.props):
                     build_it = False
                     break
 
             if build_it:
-                board.build_it = True
+                brd.build_it = True
                 if matching_term:
-                    result[matching_term].append(board.target)
-                result['all'].append(board.target)
+                    result[matching_term].append(brd.target)
+                result['all'].append(brd.target)
 
         if boards:
             remaining = set(boards) - set(found)
index 078ddf070d3d151558d8f0feec1f210162f99908..462266b4b840f0c81b496ae8eb93d25f51299af5 100644 (file)
@@ -875,11 +875,11 @@ class Builder:
         config = {}
         environment = {}
 
-        for board in boards_selected.values():
-            outcome = self.GetBuildOutcome(commit_upto, board.target,
+        for brd in boards_selected.values():
+            outcome = self.GetBuildOutcome(commit_upto, brd.target,
                                            read_func_sizes, read_config,
                                            read_environment)
-            board_dict[board.target] = outcome
+            board_dict[brd.target] = outcome
             last_func = None
             last_was_warning = False
             for line in outcome.err_lines:
@@ -894,29 +894,29 @@ class Builder:
                         if is_warning or (last_was_warning and is_note):
                             if last_func:
                                 AddLine(warn_lines_summary, warn_lines_boards,
-                                        last_func, board)
+                                        last_func, brd)
                             AddLine(warn_lines_summary, warn_lines_boards,
-                                    line, board)
+                                    line, brd)
                         else:
                             if last_func:
                                 AddLine(err_lines_summary, err_lines_boards,
-                                        last_func, board)
+                                        last_func, brd)
                             AddLine(err_lines_summary, err_lines_boards,
-                                    line, board)
+                                    line, brd)
                         last_was_warning = is_warning
                         last_func = None
-            tconfig = Config(self.config_filenames, board.target)
+            tconfig = Config(self.config_filenames, brd.target)
             for fname in self.config_filenames:
                 if outcome.config:
                     for key, value in outcome.config[fname].items():
                         tconfig.Add(fname, key, value)
-            config[board.target] = tconfig
+            config[brd.target] = tconfig
 
-            tenvironment = Environment(board.target)
+            tenvironment = Environment(brd.target)
             if outcome.environment:
                 for key, value in outcome.environment.items():
                     tenvironment.Add(key, value)
-            environment[board.target] = tenvironment
+            environment[brd.target] = tenvironment
 
         return (board_dict, err_lines_summary, err_lines_boards,
                 warn_lines_summary, warn_lines_boards, config, environment)
@@ -971,9 +971,8 @@ class Builder:
                 board.target
         """
         self._base_board_dict = {}
-        for board in board_selected:
-            self._base_board_dict[board] = Builder.Outcome(0, [], [], {}, {},
-                                                           {})
+        for brd in board_selected:
+            self._base_board_dict[brd] = Builder.Outcome(0, [], [], {}, {}, {})
         self._base_err_lines = []
         self._base_warn_lines = []
         self._base_err_line_boards = {}
@@ -1220,10 +1219,10 @@ class Builder:
             boards = []
             board_set = set()
             if self._list_error_boards:
-                for board in line_boards[line]:
-                    if not board in board_set:
-                        boards.append(board)
-                        board_set.add(board)
+                for brd in line_boards[line]:
+                    if not brd in board_set:
+                        boards.append(brd)
+                        board_set.add(brd)
             return boards
 
         def _CalcErrorDelta(base_lines, base_line_boards, lines, line_boards,
@@ -1328,7 +1327,7 @@ class Builder:
                 out_list = []
                 for line in err_lines:
                     boards = ''
-                    names = [board.target for board in line.boards]
+                    names = [brd.target for brd in line.boards]
                     board_str = ' '.join(names) if names else ''
                     if board_str:
                         out = self.col.build(colour, line.char + '(')
@@ -1549,9 +1548,9 @@ class Builder:
 
         # Get a list of boards that did not get built, if needed
         not_built = []
-        for board in board_selected:
-            if not board in board_dict:
-                not_built.append(board)
+        for brd in board_selected:
+            if not brd in board_dict:
+                not_built.append(brd)
         if not_built:
             tprint("Boards not built (%d): %s" % (len(not_built),
                   ', '.join(not_built)))
@@ -1768,7 +1767,7 @@ class Builder:
         # Create jobs to build all commits for each board
         for brd in board_selected.values():
             job = builderthread.BuilderJob()
-            job.board = brd
+            job.brd = brd
             job.commits = commits
             job.keep_outputs = keep_outputs
             job.work_in_output = self.work_in_output
index 7522ff62de6f9cda606cf58f2dbd22c7c44f7f78..6240e08c767050481de6e836b71b0bca750d639f 100644 (file)
@@ -40,7 +40,7 @@ class BuilderJob:
     """Holds information about a job to be performed by a thread
 
     Members:
-        board: Board object to build
+        brd: Board object to build
         commits: List of Commit objects to build
         keep_outputs: True to save build output files
         step: 1 to process every commit, n to process every nth commit
@@ -48,7 +48,7 @@ class BuilderJob:
             don't write to a separate output directory.
     """
     def __init__(self):
-        self.board = None
+        self.brd = None
         self.commits = []
         self.keep_outputs = False
         self.step = 1
@@ -491,7 +491,7 @@ class BuilderThread(threading.Thread):
         Returns:
             List of Result objects
         """
-        brd = job.board
+        brd = job.brd
         work_dir = self.builder.GetThreadDir(self.thread_num)
         self.toolchain = None
         if job.commits:
index fbf6706644b405c744a5f186a837a8c6b593e7c1..0265286908d4e45e96f0c6f8d9b31a0c6dd1ad4d 100644 (file)
@@ -476,12 +476,12 @@ class TestFunctional(unittest.TestCase):
         self.assertEqual(ret_code, 100)
 
         for commit in range(self._commits):
-            for board in self._boards.GetList():
-                if board.arch != 'sandbox':
-                  errfile = self._builder.GetErrFile(commit, board.target)
+            for brd in self._boards.GetList():
+                if brd.arch != 'sandbox':
+                  errfile = self._builder.GetErrFile(commit, brd.target)
                   fd = open(errfile)
                   self.assertEqual(fd.readlines(),
-                          ['No tool chain for %s\n' % board.arch])
+                          ['No tool chain for %s\n' % brd.arch])
                   fd.close()
 
     def testBranch(self):
index 27287438eeae7d6221876539fe9d21945118ed68..62ad25db397ca5c3b40938457f01514aed7dd2c4 100644 (file)
@@ -184,8 +184,8 @@ class TestBuild(unittest.TestCase):
         # TODO(sjg@chromium.org): If plus is '', we shouldn't need this
         expect += ' ' + col.build(expected_colour, plus)
         expect += '  '
-        for board in boards:
-            expect += col.build(expected_colour, ' %s' % board)
+        for brd in boards:
+            expect += col.build(expected_colour, ' %s' % brd)
         self.assertEqual(text, expect)
 
     def _SetupTest(self, echo_lines=False, threads=1, **kwdisplay_args):
index 46a4e5ed409739232931b7e60bb7d34dfcddc151..fea40ba2151a49aad3b1d4d7be1aa9b64ee8b283 100644 (file)
@@ -441,7 +441,7 @@ class Toolchains:
             args = args[:m.start(0)] + value + args[m.end(0):]
         return args
 
-    def GetMakeArguments(self, board):
+    def GetMakeArguments(self, brd):
         """Returns 'make' arguments for a given board
 
         The flags are in a section called 'make-flags'. Flags are named
@@ -462,13 +462,13 @@ class Toolchains:
         A special 'target' variable is set to the board target.
 
         Args:
-            board: Board object for the board to check.
+            brd: Board object for the board to check.
         Returns:
             'make' flags for that board, or '' if none
         """
-        self._make_flags['target'] = board.target
+        self._make_flags['target'] = brd.target
         arg_str = self.ResolveReferences(self._make_flags,
-                           self._make_flags.get(board.target, ''))
+                           self._make_flags.get(brd.target, ''))
         args = re.findall("(?:\".*?\"|\S)+", arg_str)
         i = 0
         while i < len(args):