From 859c4ed2cd41aa0ef5b5a29b3d2c64199b252ee1 Mon Sep 17 00:00:00 2001 From: Ulf Magnusson Date: Tue, 7 May 2019 10:01:36 +0200 Subject: [PATCH] west: Fix unused import and variable Making a clean slate for a pylint test in CI. '_' is a common name for non-problematic unused variables in Python. pylint knows not to flag it. Signed-off-by: Ulf Magnusson --- scripts/west_commands/zcmake.py | 2 +- scripts/west_commands/zephyr_ext_common.py | 2 -- 2 files changed, 1 insertion(+), 3 deletions(-) diff --git a/scripts/west_commands/zcmake.py b/scripts/west_commands/zcmake.py index ecd8aacb9aa..85e0b8dba5b 100644 --- a/scripts/west_commands/zcmake.py +++ b/scripts/west_commands/zcmake.py @@ -57,7 +57,7 @@ def run_cmake(args, cwd=None, capture_output=False, dry_run=False): log.dbg('Running CMake:', quote_sh_list(cmd), level=log.VERBOSE_NORMAL) p = subprocess.Popen(cmd, **kwargs) - out, err = p.communicate() + out, _ = p.communicate() if p.returncode == 0: if out: return out.decode(sys.getdefaultencoding()).splitlines() diff --git a/scripts/west_commands/zephyr_ext_common.py b/scripts/west_commands/zephyr_ext_common.py index 4edbd62602c..80f6297a6b3 100644 --- a/scripts/west_commands/zephyr_ext_common.py +++ b/scripts/west_commands/zephyr_ext_common.py @@ -8,8 +8,6 @@ Note that common helpers used by the flash and debug extension commands are in run_common -- that's for common code used by commands which specifically execute runners.''' -import os - from west import log from west.commands import WestCommand