-
Bug
-
Resolution: Fixed
-
Medium
-
None
-
None
We have a customer that has this patch segment in a patch file:
Index: utf8 =================================================================== --- utf8 (revision 0) +++ utf8 (working copy) Property changes on: utf8 ___________________________________________________________________ Added: bugtraq:label ## -0,0 +1 ## +Issue ID \ No newline at end of property Added: bugtraq:url ## -0,0 +1 ## +http://www.customer.com/blah/hello.php?helpid=%HELPID% \ No newline at end of property Added: bugtraq:message ## -0,0 +1 ## +Issue: %BUGID% \ No newline at end of property Added: bugtraq:warnifnoissue ## -0,0 +1 ## +true \ No newline at end of property Added: bugtraq:logregex ## -0,0 +1,2 ## +[Ii]ssues?:?(\s*(,|and)?\s*#\d+)+ +(\d+) \ No newline at end of property
Crucible fails to load this patch because it is not a standard hunk. JPatch fails on it too but it should give a warning instead. We should ignore/remove all of these from pre-commit patches in the short term and maybe show the permissions changes in the long term.
This issue is exactly the same as this one: http://code.google.com/p/reviewboard/issues/detail?id=2359
Workaround
Using SVN 1.8+, create the diff with the --patch-compatible flag:
svn --patch-compatible diff > patch.txt
- relates to
-
CRUC-7387 Error adding a patch ending with diffs for binary files
- Closed
-
CRUC-6925 CLI should have a better error message for svn diff with property changes and use the option "--patch-compatible" if SVN 1.8.x is detected
- Not Being Considered
- mentioned in
-
Page Loading...
-
Page Loading...
-
Page Loading...
-
Page Loading...
-
Page Loading...
-
Page Loading...
-
Page Loading...
-
Page Loading...