Re: Issue 1719 in reviewboard: post-review should support subversion changelists.
Updates: Status: Fixed Comment #3 on issue 1719 by trowb...@gmail.com: post-review should support subversion changelists. http://code.google.com/p/reviewboard/issues/detail?id=1719 This has been fixed for a while. This is currently handled with the --svn-changelist option, though that will be changing: https://reviewboard.hackpad.com/RBTools-Diff-Behavior-gRwNAvl3SzJ -- 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/groups/opt_out.
Re: Issue 1719 in reviewboard: post-review should support subversion changelists.
Updates: Labels: Component-RBTools Comment #2 on issue 1719 by trowb...@gmail.com: post-review should support subversion changelists. http://code.google.com/p/reviewboard/issues/detail?id=1719 (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-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 1719 in reviewboard: post-review should support subversion changelists.
Comment #1 on issue 1719 by joedr...@gmail.com: post-review should support subversion changelists. http://code.google.com/p/reviewboard/issues/detail?id=1719 I made this change to help a friend out recently, which reuses the --label feature. This allows the normal usage of manually listing files to diff to continue to work, and offers an alternate path. It is used like this: post-review --label name_of_svn_changelist Seems to work great! Hope this helps someone. Attachments: svn_changelist_support.diff 1.1 KB -- 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.
Issue 1719 in reviewboard: post-review should support subversion changelists.
Status: New Owner: Labels: Type-Enhancement Priority-Medium New issue 1719 by phil.cornelius: post-review should support subversion changelists. http://code.google.com/p/reviewboard/issues/detail?id=1719 *NOTE: If you have a patch, please submit it to http://reviews.reviewboard.org/ What version are you running? 1.0.5.1 What's the URL of the page this enhancement relates to, if any? Describe the enhancement and the motivation for it. Subversion v1.5 and above support changelists for associating files. This is supported in the svn diff command with --changelist option. It should be supported by post-review to allow the user to post a new review with the diffs from only those files identified by a provided changelist name. This would make the post-review tool much more useful in a subversion environment. What operating system are you using? What browser? Mac OS X, Safari Please provide any additional information below. -- 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.