Re: Issue 1517 in reviewboard: Review Board needs a better error response when using short SHA1s with git diffs

2010-09-05 Thread reviewboard

Updates:
Status: Fixed

Comment #2 on issue 1517 by chipx86: Review Board needs a better error  
response when using short SHA1s with git diffs

http://code.google.com/p/reviewboard/issues/detail?id=1517

Fixed on master (7eb6fcc)

--
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.



Re: Issue 1517 in reviewboard: Review Board needs a better error response when using short SHA1s with git diffs

2010-02-26 Thread reviewboard

Updates:
	Summary: Review Board needs a better error response when using short SHA1s  
with git diffs

Status: Confirmed
Labels: Component-SCMTools Milestone-Release1.5

Comment #1 on issue 1517 by chipx86: Review Board needs a better error  
response when using short SHA1s with git diffs

http://code.google.com/p/reviewboard/issues/detail?id=1517

This is due to git diff using short SHA1s by default. You need to either  
pass

--full-index to git diff or, preferably, use post-review.

Repurposing the bug. We really should make it clear what the problem  
actually is.


--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings

--
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.