Re: Issue 1431 in reviewboard: Need support for TortoiseSVN diffs

2011-06-02 Thread reviewboard


Comment #13 on issue 1431 by ben.al...@gmail.com: Need support for  
TortoiseSVN diffs

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

We use ReviewBoard with TortoiseSVN and have found that TSVN gives you  
multiple ways to produce a diff, but only one produces a unified diff in  
the correct format for ReviewBoard.


In general, any way of creating a diff through the TortoiseMerge utility  
(which is the default utility for viewing diffs in Tortoise) resulted in a  
diff that was not formatted correctly for ReviewBoard.  It appears that  
TortoiseMerge's diff treats the two files as simply files on your hard  
drive (not as a working copy and a pristine copy, like the svn diff  
command does) and doesn't create the header with the information that  
ReviewBoard needs.  I suspect that many other diff programs will have this  
same problem.


Instead, if you right click - TortoiseSVN submenu - Create Patch...,  
this WILL create a diff in the unified diff format expected by  
ReviewBoard.  This command appears to use the svn diff command to create  
the diff (instead of doing the comparison itself), which is what  
ReviewBoard is expecting.


In case it's important, we're using ReviewBoard 1.6beta2 and TortoiseSVN  
1.6.15



--
You received this message because you are subscribed to the Google Groups 
reviewboard-issues group.
To post to this group, send email to reviewboard-issues@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 1431 in reviewboard: Need support for TortoiseSVN diffs

2010-08-30 Thread reviewboard


Comment #12 on issue 1431 by trowbrds: Need support for TortoiseSVN diffs
http://code.google.com/p/reviewboard/issues/detail?id=1431

Issue 1608 has been merged into this issue.

--
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 1431 in reviewboard: Need support for TortoiseSVN diffs

2010-06-10 Thread reviewboard

Updates:
Labels: -Milestone-Release1.5 Milestone-Release1.6

Comment #11 on issue 1431 by trowbrds: Need support for TortoiseSVN diffs
http://code.google.com/p/reviewboard/issues/detail?id=1431

(No comment was entered for this change.)

--
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 1431 in reviewboard: Need support for TortoiseSVN diffs

2010-03-23 Thread reviewboard


Comment #10 on issue 1431 by caffeinatedcoder: Need support for TortoiseSVN  
diffs

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

Yeah I see that now.  I will have to see how it works (assuming I can get  
reviewboard
to work with https Subversion on our Windows 2003 Server machines - it  
works fine on

workstation).  The others should probably as well though :)  Thanks.

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



Re: Issue 1431 in reviewboard: Need support for TortoiseSVN diffs

2010-03-22 Thread reviewboard

Updates:
Labels: -Type-Defect Type-Enhancement

Comment #8 on issue 1431 by trowbrds: Need support for TortoiseSVN diffs
http://code.google.com/p/reviewboard/issues/detail?id=1431

(No comment was entered for this change.)

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



Re: Issue 1431 in reviewboard: Need support for TortoiseSVN diffs

2010-03-04 Thread reviewboard


Comment #7 on issue 1431 by caffeinatedcoder: Need support for TortoiseSVN  
diffs

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

There are two ways I was creating a diff.  1) You right click on the file  
and choose
diff with previous version (our organization has to post-commit reviews and  
we cannot
put the command line svn on every developer workstation) and then Save-As.   
I was not
able to get this diff to work at all.  2) You can right-click on the file ,  
choose
show log, right click on a specific entry and choose show unified diff.   
Then you
copy all the text into a separate diff file.  This adds the index line I  
mentioned

that you have to remove.

I have never tried creating a patch but I don't see an option to do this  
from the svn

context menu or the repo browser.  Where is this at?

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