Re: [oe] [meta-oe][warrior][PATCH 1/2] gitpkgv.bbclass: Use --git-dir option

2019-11-25 Thread Peter Kjellerstedt
Uhm, what do you mean with "wrong ml"? Given that the 
gitpkgpv.bbclass is in meta-oe, this should be the correct 
mailing list for at patch to it...

//Peter

> -Original Message-
> From: openembedded-devel-boun...@lists.openembedded.org  devel-boun...@lists.openembedded.org> On Behalf Of akuster808
> Sent: den 24 november 2019 17:46
> To: Niko Mauno ; openembedded-
> de...@lists.openembedded.org
> Subject: Re: [oe] [meta-oe][warrior][PATCH 1/2] gitpkgv.bbclass: Use --
> git-dir option
> 
> wrong ml
> 
> On 11/23/19 5:46 AM, Niko Mauno wrote:
> > From: "niko.ma...@vaisala.com" 
> >
> > Avoid redundant shell working directory change by resorting to
> > '--git-dir' option for git command instead.
> >
> > Signed-off-by: Niko Mauno 
> > Signed-off-by: Khem Raj 
> > ---
> >  meta-oe/classes/gitpkgv.bbclass | 11 ---
> >  1 file changed, 4 insertions(+), 7 deletions(-)
> >
> > diff --git a/meta-oe/classes/gitpkgv.bbclass b/meta-
> oe/classes/gitpkgv.bbclass
> > index 2d9680a35..22609f584 100644
> > --- a/meta-oe/classes/gitpkgv.bbclass
> > +++ b/meta-oe/classes/gitpkgv.bbclass
> > @@ -87,10 +87,8 @@ def get_git_pkgv(d, use_tags):
> >
> >  if not os.path.exists(rev_file) or
> os.path.getsize(rev_file)==0:
> >  commits = bb.fetch2.runfetchcmd(
> > -"cd %(repodir)s && "
> > -"git rev-list %(rev)s -- 2> /dev/null "
> > -"| wc -l" % vars,
> > -d, quiet=True).strip().lstrip('0')
> > +"git --git-dir=%(repodir)s rev-list %(rev)s --
> 2>/dev/null | wc -l"
> > +% vars, d, quiet=True).strip().lstrip('0')
> >
> >  if commits != "":
> >  oe.path.remove(rev_file, recurse=False)
> > @@ -105,9 +103,8 @@ def get_git_pkgv(d, use_tags):
> >  if use_tags:
> >  try:
> >  output = bb.fetch2.runfetchcmd(
> > -"cd %(repodir)s && "
> > -"git describe %(rev)s 2>/dev/null" % vars,
> > -d, quiet=True).strip()
> > +"git --git-dir=%(repodir)s describe %(rev)s
> 2>/dev/null"
> > +% vars, d, quiet=True).strip()
> >  ver = gitpkgv_drop_tag_prefix(output)
> >  except Exception:
> >  ver = "0.0-%s-g%s" % (commits, vars['rev'][:7])
> 
> --
> ___
> Openembedded-devel mailing list
> Openembedded-devel@lists.openembedded.org
> http://lists.openembedded.org/mailman/listinfo/openembedded-devel
-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


Re: [oe] [meta-oe][warrior][PATCH 1/2] gitpkgv.bbclass: Use --git-dir option

2019-11-24 Thread akuster808
wrong ml

On 11/23/19 5:46 AM, Niko Mauno wrote:
> From: "niko.ma...@vaisala.com" 
>
> Avoid redundant shell working directory change by resorting to
> '--git-dir' option for git command instead.
>
> Signed-off-by: Niko Mauno 
> Signed-off-by: Khem Raj 
> ---
>  meta-oe/classes/gitpkgv.bbclass | 11 ---
>  1 file changed, 4 insertions(+), 7 deletions(-)
>
> diff --git a/meta-oe/classes/gitpkgv.bbclass b/meta-oe/classes/gitpkgv.bbclass
> index 2d9680a35..22609f584 100644
> --- a/meta-oe/classes/gitpkgv.bbclass
> +++ b/meta-oe/classes/gitpkgv.bbclass
> @@ -87,10 +87,8 @@ def get_git_pkgv(d, use_tags):
>  
>  if not os.path.exists(rev_file) or 
> os.path.getsize(rev_file)==0:
>  commits = bb.fetch2.runfetchcmd(
> -"cd %(repodir)s && "
> -"git rev-list %(rev)s -- 2> /dev/null "
> -"| wc -l" % vars,
> -d, quiet=True).strip().lstrip('0')
> +"git --git-dir=%(repodir)s rev-list %(rev)s -- 
> 2>/dev/null | wc -l"
> +% vars, d, quiet=True).strip().lstrip('0')
>  
>  if commits != "":
>  oe.path.remove(rev_file, recurse=False)
> @@ -105,9 +103,8 @@ def get_git_pkgv(d, use_tags):
>  if use_tags:
>  try:
>  output = bb.fetch2.runfetchcmd(
> -"cd %(repodir)s && "
> -"git describe %(rev)s 2>/dev/null" % vars,
> -d, quiet=True).strip()
> +"git --git-dir=%(repodir)s describe %(rev)s 
> 2>/dev/null"
> +% vars, d, quiet=True).strip()
>  ver = gitpkgv_drop_tag_prefix(output)
>  except Exception:
>  ver = "0.0-%s-g%s" % (commits, vars['rev'][:7])

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel


[oe] [meta-oe][warrior][PATCH 1/2] gitpkgv.bbclass: Use --git-dir option

2019-11-23 Thread Niko Mauno
From: "niko.ma...@vaisala.com" 

Avoid redundant shell working directory change by resorting to
'--git-dir' option for git command instead.

Signed-off-by: Niko Mauno 
Signed-off-by: Khem Raj 
---
 meta-oe/classes/gitpkgv.bbclass | 11 ---
 1 file changed, 4 insertions(+), 7 deletions(-)

diff --git a/meta-oe/classes/gitpkgv.bbclass b/meta-oe/classes/gitpkgv.bbclass
index 2d9680a35..22609f584 100644
--- a/meta-oe/classes/gitpkgv.bbclass
+++ b/meta-oe/classes/gitpkgv.bbclass
@@ -87,10 +87,8 @@ def get_git_pkgv(d, use_tags):
 
 if not os.path.exists(rev_file) or 
os.path.getsize(rev_file)==0:
 commits = bb.fetch2.runfetchcmd(
-"cd %(repodir)s && "
-"git rev-list %(rev)s -- 2> /dev/null "
-"| wc -l" % vars,
-d, quiet=True).strip().lstrip('0')
+"git --git-dir=%(repodir)s rev-list %(rev)s -- 
2>/dev/null | wc -l"
+% vars, d, quiet=True).strip().lstrip('0')
 
 if commits != "":
 oe.path.remove(rev_file, recurse=False)
@@ -105,9 +103,8 @@ def get_git_pkgv(d, use_tags):
 if use_tags:
 try:
 output = bb.fetch2.runfetchcmd(
-"cd %(repodir)s && "
-"git describe %(rev)s 2>/dev/null" % vars,
-d, quiet=True).strip()
+"git --git-dir=%(repodir)s describe %(rev)s 
2>/dev/null"
+% vars, d, quiet=True).strip()
 ver = gitpkgv_drop_tag_prefix(output)
 except Exception:
 ver = "0.0-%s-g%s" % (commits, vars['rev'][:7])
-- 
2.20.1

-- 
___
Openembedded-devel mailing list
Openembedded-devel@lists.openembedded.org
http://lists.openembedded.org/mailman/listinfo/openembedded-devel