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 |
