Re: Issue 3874 in reviewboard: Upload diff error 207, but the file exists

2015-05-26 Thread reviewboard

Updates:
Status: NeedInfo

Comment #1 on issue 3874 by bar...@beanbaginc.com: Upload diff error 207,  
but the file exists

https://code.google.com/p/reviewboard/issues/detail?id=3874

The URL you indicated isn't one that Review Board uses. Did you mean:

api/review-requests/{review_request_id}/diffs/

?

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Re: Issue 3874 in reviewboard: Upload diff error 207, but the file exists

2015-05-26 Thread reviewboard


Comment #3 on issue 3874 by chip...@gmail.com: Upload diff error 207, but  
the file exists

https://code.google.com/p/reviewboard/issues/detail?id=3874

Also, we'll need the full RBTools output with --debug.

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Re: Issue 3874 in reviewboard: Upload diff error 207, but the file exists

2015-05-26 Thread reviewboard


Comment #4 on issue 3874 by dnep...@yelp.com: Upload diff error 207, but  
the file exists

https://code.google.com/p/reviewboard/issues/detail?id=3874

Ah, I think I was looking at an old version of the API docs (1.5).  I was  
using the `rbt` tool, so I didn't curl the url myself.


I think it would be

POST /api/review-requests/{review_request_id}/diffs/

yes

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Re: Issue 3874 in reviewboard: Upload diff error 207, but the file exists

2015-05-26 Thread reviewboard


Comment #5 on issue 3874 by dnep...@yelp.com: Upload diff error 207, but  
the file exists

https://code.google.com/p/reviewboard/issues/detail?id=3874

Full debug output of `rbt`: http://pastebin.com/7eAG5SrB

The diff for that file looks something like this  
http://pastebin.com/MDTQpvMY


--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.


Issue 3874 in reviewboard: Upload diff error 207, but the file exists

2015-05-25 Thread reviewboard

Status: New
Owner: 
Labels: Type-Defect Priority-Medium

New issue 3874 by dnep...@yelp.com: Upload diff error 207, but the file  
exists

https://code.google.com/p/reviewboard/issues/detail?id=3874

What version are you running?

2.0.12


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

/api/json/reviewrequests/{review_request_id}/diff/new/



What steps will reproduce the problem?
1. Create a git repo with some files
2. chmod +x some file, git add, git commit that file
3. git rm the file, and commit
4. post a new review using `rbt post ...`


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

I expected my review to be posted with a diff. Instead I received


ERROR: Error uploading diff

The file was not found in the repository. (HTTP 400, API Error 207)



What operating system are you using? What browser?

N/A, issue is with the API


Please provide any additional information below.


`rbt` wasn't properly returning the file from the response, after hacking  
it up to print the full response I saw the error was caused by a file that  
I had run `chmod +x` on in one commit, and  `git rm` in the next.


I then rebased out the `chmod +x` commit, and the upload diff succeeded.

--
You received this message because this project is configured to send all  
issue notifications to this address.

You may adjust your notification preferences at:
https://code.google.com/hosting/settings

--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To unsubscribe from this group and stop receiving emails from it, send an email 
to reviewboard-issues+unsubscr...@googlegroups.com.
To post to this group, send email to reviewboard-issues@googlegroups.com.
Visit this group at http://groups.google.com/group/reviewboard-issues.
For more options, visit https://groups.google.com/d/optout.