[oe] [meta-oe][PATCH] gitpkgv: ensure that we close our opened files
Christopher Larson
chris_larson at mentor.com
Fri Jun 3 22:17:38 UTC 2016
Ignore this, just noticed the fix is already on master-next.
On Fri, Jun 3, 2016 at 11:36 AM, Christopher Larson <kergoth at gmail.com>
wrote:
> From: Christopher Larson <chris_larson at mentor.com>
>
> Avoids warnings like this with python3:
>
> WARNING:
> .../meta-openembedded/meta-filesystems/recipes-filesystems/smbnetfs/
> smbnetfs_git.bb: <string>:94: ResourceWarning: unclosed file
> <_io.TextIOWrapper
> name='.../build/downloads/git2/smbnetfs.git.sourceforge.net.gitroot.smbnetfs.smbnetfs/oe-gitpkgv_ace1c519d4'
> mode='r' encoding='UTF-8'>
>
> Signed-off-by: Christopher Larson <chris_larson at mentor.com>
> ---
> meta-oe/classes/gitpkgv.bbclass | 6 ++++--
> 1 file changed, 4 insertions(+), 2 deletions(-)
>
> diff --git a/meta-oe/classes/gitpkgv.bbclass
> b/meta-oe/classes/gitpkgv.bbclass
> index 1cba00c..4866fac 100644
> --- a/meta-oe/classes/gitpkgv.bbclass
> +++ b/meta-oe/classes/gitpkgv.bbclass
> @@ -87,11 +87,13 @@ def get_git_pkgv(d, use_tags):
>
> if commits != "":
> oe.path.remove(rev_file, recurse=False)
> - open(rev_file, "w").write("%d\n" % int(commits))
> + with open(rev_file, "w") as f:
> + f.write("%d\n" % int(commits))
> else:
> commits = "0"
> else:
> - commits = open(rev_file, "r").readline(128).strip()
> + with open(rev_file, "r") as f:
> + commits = f.readline(128).strip()
>
> if use_tags:
> try:
> --
> 2.8.0
>
>
--
Christopher Larson
kergoth at gmail dot com
Founder - BitBake, OpenEmbedded, OpenZaurus
Maintainer - Tslib
Senior Software Engineer, Mentor Graphics
More information about the Openembedded-devel
mailing list