diff options
author | Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-05-09 17:05:58 +0100 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2013-05-09 22:28:04 +0100 |
commit | 566628d8cd674a964d5824391cfd1585a1a22a87 (patch) | |
tree | 670366ee1492ff8bb18b9261dfab9d06c06b0a2d /meta/classes/imagetest-qemu.bbclass | |
parent | d2ef952851d9ef16875fdbbbc6ae6eb6cfc10cc0 (diff) | |
download | poky-566628d8cd674a964d5824391cfd1585a1a22a87.tar.gz |
class/lib: Fix up various file access methods
There are various bits of cruft that have built up around our file accesses. This patch
cleans some of them up, specifically:
* Remove pointless "from __builtin__ import file"
* Use open(), not file()
* Wrap file usage in a with container to ensure files are closed
* Add missing .close() calls in some cases
(From OE-Core rev: a43e0a8ecd0441131e929daf998c3cd454d9c8f3)
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/classes/imagetest-qemu.bbclass')
-rw-r--r-- | meta/classes/imagetest-qemu.bbclass | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/meta/classes/imagetest-qemu.bbclass b/meta/classes/imagetest-qemu.bbclass index 63ba08718e..c30d1cbc85 100644 --- a/meta/classes/imagetest-qemu.bbclass +++ b/meta/classes/imagetest-qemu.bbclass | |||
@@ -146,6 +146,7 @@ def qemuimagetest_main(d): | |||
146 | if not os.path.isfile(fulltestcase): | 146 | if not os.path.isfile(fulltestcase): |
147 | raise bb.build.FuncFailed("Testcase %s not found" % fulltestcase) | 147 | raise bb.build.FuncFailed("Testcase %s not found" % fulltestcase) |
148 | list.append((item, casefile, fulltestcase)) | 148 | list.append((item, casefile, fulltestcase)) |
149 | f.close() | ||
149 | final_list = check_list(list) | 150 | final_list = check_list(list) |
150 | return final_list | 151 | return final_list |
151 | 152 | ||