diff options
author | chipx86 <chipx86@5efc13c4-1f27-0410-8691-ff2d1f55687e> | 2009-05-19 07:06:48 +0000 |
---|---|---|
committer | chipx86 <chipx86@5efc13c4-1f27-0410-8691-ff2d1f55687e> | 2009-05-19 07:06:48 +0000 |
commit | 73b55bf3afefcc24d3a7d8f382de4a7cc1bc307c (patch) | |
tree | c43e12bdf2bdc87aa4a8cfaf22a363be3e88924f | |
parent | 36f2eaa22a137e17552916d8cd4d0e39fd8d9f58 (diff) |
Fix the ClearCaseClient to not break post-review if cleartool isn't installed. We now properly do a check_install on "clearcase help".
-rwxr-xr-x | scripts/post-review | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/scripts/post-review b/scripts/post-review index ef51323..405bed6 100755 --- a/scripts/post-review +++ b/scripts/post-review @@ -739,6 +739,9 @@ class ClearCaseClient(SCMClient): return md5(fname).hexdigest() def get_repository_info(self): + if not check_install('cleartool help'): + return None + # We must be running this from inside a view. # Otherwise it doesn't make sense. self.viewinfo = execute(["cleartool", "pwv", "-short"]) |