You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@hbase.apache.org by bu...@apache.org on 2018/02/28 17:58:47 UTC
[3/3] hbase git commit: HBASE-20106 [api compliance chacker] Fix Bug
Where Branch Isn't Found
HBASE-20106 [api compliance chacker] Fix Bug Where Branch Isn't Found
While git rev-parse, sometimes the branch cannot be found unless
the remote is specified. This fix tries to use "origin" if the
remote is not specified and the branch is not found.
Signed-off-by: Sean Busbey <bu...@apache.org>
Project: http://git-wip-us.apache.org/repos/asf/hbase/repo
Commit: http://git-wip-us.apache.org/repos/asf/hbase/commit/ecc4c87e
Tree: http://git-wip-us.apache.org/repos/asf/hbase/tree/ecc4c87e
Diff: http://git-wip-us.apache.org/repos/asf/hbase/diff/ecc4c87e
Branch: refs/heads/branch-1.3
Commit: ecc4c87ef373fd1b99350a34b17f2b0a56cf2c6f
Parents: d74f8f0
Author: Alex Leblang <al...@cloudera.com>
Authored: Thu Sep 7 12:31:53 2017 -0400
Committer: Sean Busbey <bu...@apache.org>
Committed: Wed Feb 28 11:57:33 2018 -0600
----------------------------------------------------------------------
dev-support/checkcompatibility.py | 7 ++++++-
1 file changed, 6 insertions(+), 1 deletion(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/hbase/blob/ecc4c87e/dev-support/checkcompatibility.py
----------------------------------------------------------------------
diff --git a/dev-support/checkcompatibility.py b/dev-support/checkcompatibility.py
index cad17e8..ad1f774 100755
--- a/dev-support/checkcompatibility.py
+++ b/dev-support/checkcompatibility.py
@@ -116,7 +116,12 @@ def checkout_java_tree(rev, path):
def get_git_hash(revname):
""" Convert 'revname' to its SHA-1 hash. """
- return check_output(["git", "rev-parse", revname],
+ try:
+ return check_output(["git", "rev-parse", revname],
+ cwd=get_repo_dir()).strip()
+ except:
+ revname = "origin/" + revname
+ return check_output(["git", "rev-parse", revname],
cwd=get_repo_dir()).strip()