Labels: Type-Defect Priority-Medium
New issue 2338 by drenna...@gmail.com: post-review --revision-range does
not work for clearcase
* NOTE: Do not post confidential information in this bug report. *
* If you need immediate support, please contact *
* reviewbo...@googlegroups.com *
What version are you running?
What's the URL of the page containing the problem?
What steps will reproduce the problem?
..try with a single revision..
...or multiple revisions..
What is the expected output? What do you see instead?
Instead of the diffs being generated I get an error saying
"There don't seem to be any diffs!"
What operating system are you using? What browser?
Please provide any additional information below.
The problem is the diff_between_revisions code in the ClearCaseClient.
According to the docs at
format of --revision-range should be STARTREV:ENDREV
However the code is splitting based on ';' not ':'
1485 def diff_between_revisions(self, revision_range, args,
1486 """Performs a diff between passed revisions or branch."""
1488 # Convert revision range to list of:
1489 # (previous version, current version) tuples
1490 revision_range = revision_range.split(';')
1491 changeset = zip(revision_range[0::2], revision_range[1::2])
1493 return (self.do_diff(changeset), None)
The other thing is the docs don't mention how to format --revision-range
for more than one revision. Doing
--revision-range=fileAOLDREV:fileANEWREV:fileBOLDREV:fileBNEWREV works but
As a workaround for this bug I have to use ';' to separate the revisions..
You received this message because you are subscribed to the Google Groups
To post to this group, send email to email@example.com.
To unsubscribe from this group, send email to
For more options, visit this group at