Sorry for my double post. Please remove issue #1522.
What version are you running?
* 1.5 beta1
What's the URL of the page containing the problem?
* intranet URL
What steps will reproduce the problem?
1. commit a file including a string '$Id:'
2. post a review with that file
reviewboard puts an error message of an error of 'patch' command.
What is the expected output? What do you see instead?
Correct diffs.
What operating system are you using? What browser?
Server: Ubuntu 8.04.01
Please provide any additional information below.
Here is a patch:
Index: reviewboard/scmtools/svn.py
===================================================================
--- reviewboard/scmtools/svn.py (revision 32)
+++ reviewboard/scmtools/svn.py (working copy)
@@ -169,7 +169,7 @@
for name in keyword_str.split(" ")
for keyword in self.keywords.get(name, [])]
- return re.sub(r"\$(%s):(:?)([^\$]+)\$" % '|'.join(keywords),
+ return re.sub(r"\$(%s):(:?)([^\$\n\r]+)\$" % '|'.join(keywords),
repl, data)
The original regex matches multiple lines if subversion's keyword
substitution doesn't end with '$' by mistake.
This patch works for me because I think the keyword substitution shouldn't
extend over multiple lines, but I'm not sure other people agree with me.
Thank you