[oe-commits] Chris Larson : metadata_scm: use rev-parse rather than show-ref
git version control
git at git.openembedded.org
Mon Nov 8 15:08:05 UTC 2010
Module: openembedded.git
Branch: darwin
Commit: a3d8c448c2bb3ea2c9a92bdb442fb4b39144bae0
URL: http://gitweb.openembedded.net/?p=openembedded.git&a=commit;h=a3d8c448c2bb3ea2c9a92bdb442fb4b39144bae0
Author: Chris Larson <chris_larson at mentor.com>
Date: Thu Nov 4 04:18:02 2010 +0000
metadata_scm: use rev-parse rather than show-ref
show-ref will show all matching refs, so a "show-ref HEAD" will show not just
the local HEAD, but the fetched remote ones as well. This isn't what we want
for this function, so change it to use rev-parse with --verify, and also
change it to use --short, to shorten the hash to a more palatable length.
Signed-off-by: Chris Larson <chris_larson at mentor.com>
Acked-by: Tim Harvey <harvey.tim at gmail.com>
Acked-by: Khem Raj <raj.khem at gmail.com>
---
classes/metadata_scm.bbclass | 22 +++++++++++++---------
1 files changed, 13 insertions(+), 9 deletions(-)
diff --git a/classes/metadata_scm.bbclass b/classes/metadata_scm.bbclass
index ffc6a8a..f79ea19 100644
--- a/classes/metadata_scm.bbclass
+++ b/classes/metadata_scm.bbclass
@@ -63,14 +63,18 @@ def base_get_metadata_svn_revision(path, d):
return revision
def base_get_metadata_git_branch(path, d):
- branch = os.popen('cd %s; PATH=%s git symbolic-ref HEAD 2>/dev/null' % (path, d.getVar("PATH", 1))).read().rstrip()
-
- if len(branch) != 0:
- return branch.replace("refs/heads/", "")
- return "<unknown>"
+ try:
+ rev = oe_run(d, ["git", "symbolic-ref", "HEAD"], cwd=path).rstrip()
+ except oe.process.CmdError:
+ rev = "<unknown>"
+ else:
+ rev = rev.replace("refs/heads/", "")
+ return rev
def base_get_metadata_git_revision(path, d):
- rev = os.popen("cd %s; PATH=%s git show-ref HEAD 2>/dev/null" % (path, d.getVar("PATH", 1))).read().split(" ")[0].rstrip()
- if len(rev) != 0:
- return rev
- return "<unknown>"
+ try:
+ rev = oe_run(d, ["git", "rev-parse", "--verify", "--short", "HEAD"],
+ cwd=path).rstrip()
+ except oe.process.CmdError:
+ rev = "<unknown>"
+ return rev
More information about the Openembedded-commits
mailing list