diff options
author | Ed Bartosh <ed.bartosh@linux.intel.com> | 2015-09-02 13:58:16 +0300 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2015-09-03 12:43:31 +0100 |
commit | 791a3912d98014105bdb2a8585e4a1b7ae8120b1 (patch) | |
tree | bcfb06b795482f6113cfa6b5c967a0bbb9b4bc1c /scripts/lib/wic/utils/runner.py | |
parent | 14b47e22f9b2566320ab6ef103da1501bca129de (diff) | |
download | poky-791a3912d98014105bdb2a8585e4a1b7ae8120b1.tar.gz |
wic: fix short variable names
Made short variable names longer and more readable.
Fixed pylint warnings "Invalid variable name" and
"Invalid argument name".
(From OE-Core rev: 872cb0d5d79b26f34e6b35d7be8870d245021be4)
Signed-off-by: Ed Bartosh <ed.bartosh@linux.intel.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'scripts/lib/wic/utils/runner.py')
-rw-r--r-- | scripts/lib/wic/utils/runner.py | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/scripts/lib/wic/utils/runner.py b/scripts/lib/wic/utils/runner.py index f6a1d0828c..7431917ff0 100644 --- a/scripts/lib/wic/utils/runner.py +++ b/scripts/lib/wic/utils/runner.py | |||
@@ -62,13 +62,13 @@ def runtool(cmdln_or_args, catch=1): | |||
62 | serr = subprocess.STDOUT | 62 | serr = subprocess.STDOUT |
63 | 63 | ||
64 | try: | 64 | try: |
65 | p = subprocess.Popen(cmdln_or_args, stdout=sout, | 65 | process = subprocess.Popen(cmdln_or_args, stdout=sout, |
66 | stderr=serr, shell=shell) | 66 | stderr=serr, shell=shell) |
67 | (sout, serr) = p.communicate() | 67 | (sout, serr) = process.communicate() |
68 | # combine stdout and stderr, filter None out | 68 | # combine stdout and stderr, filter None out |
69 | out = ''.join(filter(None, [sout, serr])) | 69 | out = ''.join(filter(None, [sout, serr])) |
70 | except OSError, e: | 70 | except OSError, err: |
71 | if e.errno == 2: | 71 | if err.errno == 2: |
72 | # [Errno 2] No such file or directory | 72 | # [Errno 2] No such file or directory |
73 | msger.error('Cannot run command: %s, lost dependency?' % cmd) | 73 | msger.error('Cannot run command: %s, lost dependency?' % cmd) |
74 | else: | 74 | else: |
@@ -77,12 +77,12 @@ def runtool(cmdln_or_args, catch=1): | |||
77 | if catch != 3: | 77 | if catch != 3: |
78 | os.close(dev_null) | 78 | os.close(dev_null) |
79 | 79 | ||
80 | return (p.returncode, out) | 80 | return (process.returncode, out) |
81 | 81 | ||
82 | def show(cmdln_or_args): | 82 | def show(cmdln_or_args): |
83 | # show all the message using msger.verbose | 83 | # show all the message using msger.verbose |
84 | 84 | ||
85 | rc, out = runtool(cmdln_or_args, catch=3) | 85 | rcode, out = runtool(cmdln_or_args, catch=3) |
86 | 86 | ||
87 | if isinstance(cmdln_or_args, list): | 87 | if isinstance(cmdln_or_args, list): |
88 | cmd = ' '.join(cmdln_or_args) | 88 | cmd = ' '.join(cmdln_or_args) |
@@ -100,7 +100,7 @@ def show(cmdln_or_args): | |||
100 | msg += '\n +----------------' | 100 | msg += '\n +----------------' |
101 | 101 | ||
102 | msger.verbose(msg) | 102 | msger.verbose(msg) |
103 | return rc | 103 | return rcode |
104 | 104 | ||
105 | def outs(cmdln_or_args, catch=1): | 105 | def outs(cmdln_or_args, catch=1): |
106 | # get the outputs of tools | 106 | # get the outputs of tools |