Comment #4 on issue 507 by thornsoft.chris: Add an option to paste in a diff instead of uploading for cvs users http://code.google.com/p/reviewboard/issues/detail?id=507
I love this idea.... Paste a Unidiff from TSvn, etc.. right into the review request box. Have some regex rules for stream editing the diff to make it fit. In our case (subversion with Tortoise), the diffs from Tortoise are nearly correct, but require a little editing of the paths at the top. ex: change c:\project\2009 to \2009 This would save a few steps. And with some simple regex/stream editing, we should be able to cook up rules to fix most diffs, on-the-fly. Also, in our case, post-review isn't a good solution. We typically have several files in various directories that need to go into the same review, yet there are files in those directories that have changed, but are NOT part of the review. So we need to make a UniDiff that spans several directories, with just the files that we want. Tortoise can make these Diffs easily, and ReviewBoard handles them nicely (multiple files from different directories, in a single diff). -- 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-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 -~----------~----~----~----~------~----~------~--~---