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 --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 -- 2.43.0