diff options
author | Christian Hammond <chipx86@chipx86.com> | 2010-02-21 15:29:55 -0800 |
---|---|---|
committer | Christian Hammond <chipx86@chipx86.com> | 2010-02-21 15:29:55 -0800 |
commit | 24b1e9d1193ed200aeae303b33616a9d02bd6213 (patch) | |
tree | e74df0a5d2b4530b60dbfb058749243f8942f890 | |
parent | c45022115591a10f887bedf9d079a808ae7623e1 (diff) |
Fix a crash when using ClearCase.
The ClearCase code was calling a command and expecting the result split
across two variables, but didn't pass split_lines=True. Now it does!
Patch by Bryan Halter
Fixes bug #1423
-rw-r--r-- | AUTHORS | 1 | ||||
-rwxr-xr-x | rbtools/postreview.py | 2 |
2 files changed, 2 insertions, 1 deletions
@@ -7,6 +7,7 @@ Lead Developers: Contributors: * Anthony Cruz + * Bryan Halter * Chris Clark * Dan Savilonis * Dana Lacoste diff --git a/rbtools/postreview.py b/rbtools/postreview.py index 207f8be..91adb50 100755 --- a/rbtools/postreview.py +++ b/rbtools/postreview.py @@ -847,7 +847,7 @@ class ClearCaseClient(SCMClient): # Call cleartool to get this version and the previous version # of the element. curr_version, pre_version = execute( - ["cleartool", "desc", "-pre", elem_path]) + ["cleartool", "desc", "-pre", elem_path], split_lines=True) curr_version = cpath.normpath(curr_version) pre_version = pre_version.split(':')[1].strip() |