diff options
| author | Simon Glass | 2022-07-11 19:03:57 -0600 |
|---|---|---|
| committer | Tom Rini | 2022-08-05 11:47:55 -0400 |
| commit | f4ed4706ef6668890bfc906ea3071429fb7f8990 (patch) | |
| tree | aaedf63d83d5af5ef48592bcafcf8bd49bc839d5 /tools/buildman/test.py | |
| parent | ae1a09f8039a064135629e3b37e108e8ee1814b3 (diff) | |
buildman: Avoid using board as a variable
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>
Diffstat (limited to 'tools/buildman/test.py')
| -rw-r--r-- | tools/buildman/test.py | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/tools/buildman/test.py b/tools/buildman/test.py index 27287438eea..62ad25db397 100644 --- a/tools/buildman/test.py +++ b/tools/buildman/test.py @@ -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): |