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:25:31 +0100 |
commit | a43e0a8ecd0441131e929daf998c3cd454d9c8f3 (patch) | |
tree | 69be538a129e3f9e985aa65d07005bf067371aa5 /meta/classes/imagetest-qemu.bbclass | |
parent | 4808629faa1222b31f92a3e410e06adb8e081293 (diff) | |
download | openembedded-core-contrib-a43e0a8ecd0441131e929daf998c3cd454d9c8f3.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
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): if not os.path.isfile(fulltestcase): raise bb.build.FuncFailed("Testcase %s not found" % fulltestcase) list.append((item, casefile, fulltestcase)) + f.close() final_list = check_list(list) return final_list |