Summary:

Make RB understand $Header$ keyword in Subversion

Review Request #3104 — Created May 11, 2012 and submitted May 20, 2012, 2:26 p.m.

Information
Owner:	stilor
Repository:	Review Board
Branch:
Bugs:
Depends On:
Reviewers
Groups:	reviewboard
People:

Description

RB fails to display a diff ("The patch to 'foo.c' didn't apply cleanly.") if the patch involves changes on the line containing $Header$ keyword. The reason is that SVNTool does not know about the $Header$ keyword and thus does not collapse it when preparing the original file for patching. $Header$ is standard SVN keyword, see here:

http://svnbook.red-bean.com/en/1.6/svn.advanced.props.special.keywords.html

Testing Done

./reviewboard/manage.py test

Expand All
Collapse All

Change Summary:

Rerun the tests with bzr installed, no more failures.

Testing Done:

		./reviewboard/manage.py test
-
-		3 errors with Bzr repository tests, same as before the changes.

Ship it!

```
Ship It!
```

Status: Closed (submitted)

Change Summary:

Pushed to release-1.6.x as 2517730. Thanks!