diff options
| author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-05-09 21:06:45 +0000 |
|---|---|---|
| committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-05-09 22:28:04 +0100 |
| commit | 4a081b5a52e3d27da8d4b062f3fda292e8d8fb0a (patch) | |
| tree | a555b39b41e4ec36c212481fcd2887cde2ee30dd /bitbake/lib/bb/process.py | |
| parent | 7f2bf08280f11daa002f4a9e870c2b77711cbf90 (diff) | |
| download | poky-4a081b5a52e3d27da8d4b062f3fda292e8d8fb0a.tar.gz | |
bitbake: lib: Clean up various file access syntax
Python 3 is stricter about how files are accessed. Specficially:
* Use open(), not file()
* Use binary mode for binary files (when checksumming)
* Use with statements to ensure files get closed
* Add missing file close statements
(Bitbake rev: 9f08b901375ba640f47596f1bcf43f98a931550f)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'bitbake/lib/bb/process.py')
| -rw-r--r-- | bitbake/lib/bb/process.py | 4 |
1 files changed, 4 insertions, 0 deletions
diff --git a/bitbake/lib/bb/process.py b/bitbake/lib/bb/process.py index 05b51725f1..1aeec788c1 100644 --- a/bitbake/lib/bb/process.py +++ b/bitbake/lib/bb/process.py | |||
| @@ -102,6 +102,10 @@ def _logged_communicate(pipe, log, input): | |||
| 102 | log.write(data) | 102 | log.write(data) |
| 103 | finally: | 103 | finally: |
| 104 | log.flush() | 104 | log.flush() |
| 105 | if pipe.stdout is not None: | ||
| 106 | pipe.stdout.close() | ||
| 107 | if pipe.stderr is not None: | ||
| 108 | pipe.stderr.close() | ||
| 105 | return ''.join(outdata), ''.join(errdata) | 109 | return ''.join(outdata), ''.join(errdata) |
| 106 | 110 | ||
| 107 | def run(cmd, input=None, log=None, **options): | 111 | def run(cmd, input=None, log=None, **options): |
