Issue 726:

Comment #1 by bengchuan79:
Pardon me for doing it again. Anyway, this is the bug report

The source of the problem is caused within "_depot_to_local(self,  
line 1060:

return last_line.split(' ')[2]

The last_line.split(' ')[2] just happens to have "\n" appending at the  
back. This
causes the diff header cannot be read properly.

The fix I did was removing the "\n" at in the last_line.split(' ')[2] before
returning the string:

# XXX: This breaks on filenames with spaces
filename = last_line.split(' ')[2]
filename = filename.replace('\n', '')
return filename

Again, terribly sorry about the mistake I made

You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:

You received this message because you are subscribed to the Google Groups 
"reviewboard-issues" group.
To post to this group, send email to
To unsubscribe from this group, send email to [EMAIL PROTECTED]
For more options, visit this group at

Reply via email to