1425: post-review + subversion + empty files produces diffs that Review Board cannot handle

ndrt*****@gmai***** (Google Code) (Is this you? Claim this profile.)
Feb. 16, 2014
895
What version are you running?
1.1 Alpha 2 in a post-commit setup

What's the URL of the page containing the problem?
Private

What steps will reproduce the problem?
1. ./post-commit /var/lib/svn/repo 14
2.
3.

What is the expected output? What do you see instead?
Successful review

What operating system are you using? What browser?
CentOS 5.2 + subversion 1.5.7

Please provide any additional information below.


Problem appears to be the diff generated. The revision is only the deletion
of empty ".sf" files.

Command ran by post-review to obtain diff:

svn diff --diff-cmd=diff file:///var/lib/svn/repo/@13
file:///var/lib/svn/repo/@14

Output of diff:

Index: //trunk/test/unit/.sf
===================================================================
Index: //trunk/test/functional/.sf
===================================================================
Index: //trunk/test/functional/frontend/.sf
===================================================================


Due to the ambiguity of the diff, there might be a few possible work-arounds:

1. Change subversion command to include "--no-diff-deleted" which adds
"(deleted)" to the diff Index line.
2. Add a diff wrapper command that will convert empty diffs to
understandable diffs.
3. Change ReviewBoard to accept/understand empty diffs which signifies a
deletion or addition depending on whether the file exists or not.

Thanks

Rob O.
Ender Tech Corp.
www.endertech.com
#1 ndrt*****@gmai***** (Google Code) (Is this you? Claim this profile.)
I forgot the error message:

Error uploading diff: One or more fields had errors (105)
Your review request still exists, but the diff is not attached.
chipx86
#2 chipx86
  • +Confirmed
  • +Component-DiffParser
    +Milestone-Release1.0.x
chipx86
#3 chipx86
Probably best fixed in post-review. There's just not enough context to do the right
thing on Review Board's side.
  • -Component-DiffParser
    -Milestone-Release1.0.x
    +Component-RBTools
    +Milestone-RBTools-Release1.0
david
#4 david
  • -Milestone-RBTools-Release1.0
  • +post-review + subversion + empty files produces diffs that Review Board cannot handle
david
#5 david