Pylint for common/runner.py

C: 87, 0: Exactly one space required after assignment
msg =  'running command: "%s"' % cmd
        ^ (bad-whitespace)
W: 80, 4: Attempting to unpack a non-sequence defined at line 40 (unpacking-non-sequence)

Change-Id: I30de9764b6df5792f6c0fff297b71e7ad09df361

diff --git a/common/runner.py b/common/runner.py
index 649c0e1..f895ffb 100644 (file)
--- a/common/runner.py
+++ b/common/runner.py
@@ -37,7 +37,7 @@ def runtool(cmdln_or_args, catch=1):
 
     if catch not in (0, 1, 2, 3):
         # invalid catch selection, will cause exception, that's good
-        return None
+        return (None, None)
 
     if isinstance(cmdln_or_args, list):
         cmd, shell = cmdln_or_args[0], False
@@ -84,7 +84,7 @@ def show(cmdln_or_args):
     else:
         cmd = cmdln_or_args
 
-    msg =  'running command: "%s"' % cmd
+    msg = 'running command: "%s"' % cmd
     if out and out.strip():
         msg += ', with output::'
         msg += '\n  +----------------'