diff mbox series

[Concept,11/22] buildman: Remove unused variables in test.py

Message ID 20260106142834.2511220-12-sjg@u-boot.org
State New
Headers
Series buildman: Clean up test.py for pylint compliance |

Commit Message

Simon Glass Jan. 6, 2026, 2:28 p.m. UTC 
  From: Simon Glass <simon.glass@canonical.com>

Remove unused stdout/stderr from terminal.capture() context and
remove unused diff_paths variable to fix pylint W0612 warnings.

Co-developed-by: Claude <noreply@anthropic.com>
Signed-off-by: Simon Glass <simon.glass@canonical.com>
---

 tools/buildman/test.py | 3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)
diff mbox series

Patch

diff --git a/tools/buildman/test.py b/tools/buildman/test.py
index eae614b56b5..31458c35f57 100644
--- a/tools/buildman/test.py
+++ b/tools/buildman/test.py
@@ -634,7 +634,7 @@  class TestBuild(unittest.TestCase):
     def test_toolchain_download(self):
         """Test that we can download toolchains"""
         if use_network:
-            with terminal.capture() as (stdout, stderr):
+            with terminal.capture():
                 url = self.toolchains.locate_arch_url('arm')
             self.assertRegex(url, 'https://www.kernel.org/pub/tools/'
                     'crosstool/files/bin/x86_64/.*/'
@@ -1000,7 +1000,6 @@  class TestBuild(unittest.TestCase):
         new_path = None
         if b'PATH' in diff:
             new_path = diff[b'PATH'].split(b':')
-            diff_paths = [p for p in new_path if p not in orig_path]
             diff_path = b':'.join(p for p in new_path if p not in orig_path)
             if diff_path:
                 diff[b'PATH'] = diff_path