diff mbox series

[Concept,06/15] buildman: Fix pylint warning in test_skip_dtc()

Message ID 20250819193918.874215-7-sjg@u-boot.org
State New
Headers
Series test: Improvements to make check et al |

Commit Message

Simon Glass Aug. 19, 2025, 7:39 p.m. UTC 
  From: Simon Glass <sjg@chromium.org>

Use a different variable to avoid overriding the toolchain module in
this function.

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

 tools/buildman/test.py | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)
diff mbox series

Patch

diff --git a/tools/buildman/test.py b/tools/buildman/test.py
index c7000b58bdf..b4e612bc5d6 100644
--- a/tools/buildman/test.py
+++ b/tools/buildman/test.py
@@ -1030,14 +1030,14 @@  class TestBuild(unittest.TestCase):
 
             build = builder.Builder(self.toolchains, self.base_dir, None, 0, 2,
                                     dtc_skip=True)
-            toolchain = self.toolchains.Select('arm')
-            env = build.make_environment(toolchain)
+            tch = self.toolchains.Select('arm')
+            env = build.make_environment(tch)
             self.assertIn(b'DTC', env)
 
             # Try the normal case, i.e. not skipping the dtc build
             build = builder.Builder(self.toolchains, self.base_dir, None, 0, 2)
-            toolchain = self.toolchains.Select('arm')
-            env = build.make_environment(toolchain)
+            tch = self.toolchains.Select('arm')
+            env = build.make_environment(tch)
             self.assertNotIn(b'DTC', env)
         finally:
             os.environ['PATH'] = old_path