From 24b1e9d1193ed200aeae303b33616a9d02bd6213 Mon Sep 17 00:00:00 2001 From: Christian Hammond Date: Sun, 21 Feb 2010 15:29:55 -0800 Subject: 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 --- AUTHORS | 1 + rbtools/postreview.py | 2 +- 2 files changed, 2 insertions(+), 1 deletion(-) diff --git a/AUTHORS b/AUTHORS index 15c0ad4..928fb95 100644 --- a/AUTHORS +++ b/AUTHORS @@ -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() -- cgit v1.2.3