[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