Status: New
Owner: ----
Labels: Type-Defect Priority-Medium
New issue 1792 by dmitry.murashenkov: post-review fails to upload diff with
only binary files modified
http://code.google.com/p/reviewboard/issues/detail?id=1792
What version are you running?
1.0.9
What steps will reproduce the problem?
1. Add new binary file to SVN repository
2. Commit it
3. Create diff using svnlook diff -r 2 $REPOS
4. Upload diff using post-review --diff-filename=$DIFF_FILE
Expected result: upload successfull, UI says that some binary file was
added (as it does if using post-review --revision-range=1:2)
Actual result: Error message:
The generated diff file was empty. This usually means no files were
modified in this change.
Diff-file content:
Added: trunk/mail4.jar
===================================================================
(Binary files differ)
Property changes on: trunk/mail4.jar
___________________________________________________________________
Added: svn:mime-type
+ application/octet-stream
Environment:
Ubuntu 10.4
SVN 1.6.6dfsg-2ubuntu1
If post-review run with --debug option:
Error uploading diff
The generated diff file was empty. This usually means no files were
modified in this change.
Try running with --output-diff and --debug for more information.
svn info file:///home/raketa/SVN-Test
diff --version
repository info: Path: file:///home/raketa/SVN-Test, Base path: /,
Supports changesets: False
Looking for 'localhost /' cookie in .post-review-cookies.txt
Cookie file loaded, but no cookie for this server
==> Review Board Login Required
Enter username and password for Review Board at http://localhost/
Logging in with username "admin"
HTTP POSTing to http://localhost/api/json/accounts/login/:
{'username': 'admin', 'password': '**************'}
Logged in.
HTTP GETting /api/json/repositories/
HTTP GETting /api/json/repositories/1/info/
repository info: Path: file:///home/raketa/SVN-Test, Base path: /,
Supports changesets: False
Attempting to create review request on file:///home/raketa/SVN-Test for
None
Submitting the review request as raketa
HTTP POSTing to http://localhost/api/json/reviewrequests/new/:
{'repository_path':
u'file:///home/raketa/SVN-Test', 'submit_as': 'raketa'}
Review request created
Attempting to set field 'target_groups' to 'Test' for review
request '56'
HTTP POSTing to http://localhost/api/json/reviewrequests/56/draft/set/:
{'target_groups': 'Test'}
Attempting to set field 'summary' to 'abc' for review request '56'
HTTP POSTing to http://localhost/api/json/reviewrequests/56/draft/set/:
{'summary': 'abc'}
Attempting to set field 'description' to 'Revision 91' for review
request '56'
HTTP POSTing to http://localhost/api/json/reviewrequests/56/draft/set/:
{'description': 'Revision 91'}
Uploading diff, size: 272
HTTP POSTing to http://localhost/api/json/reviewrequests/56/diff/new/:
{'basedir': '/'}
Got API Error 105 (HTTP code 200): One or more fields had errors
Error data: {u'fields': {u'path': [u'The diff file is empty']},
u'stat': u'fail', u'err': {u'msg': u'One or more fields had errors',
u'code': 105}}
Your review request still exists, but the diff is not attached.
--
You received this message because you are subscribed to the Google Groups
"reviewboard-issues" group.
To post to this group, send email to reviewboard-iss...@googlegroups.com.
To unsubscribe from this group, send email to
reviewboard-issues+unsubscr...@googlegroups.com.
For more options, visit this group at
http://groups.google.com/group/reviewboard-issues?hl=en.