Hi Skip,

Your versions of post-review and Review Board sound pretty out-of-date,
first of all, so I'd recommend setting aside some time to do an upgrade.

Let me see if I have this right. You have a directory full of committed code
that hasn't been reviewed yet. This code may span multiple revisions. You
want to put that code up for review. Is that correct?

There's no built-in option for doing this with CVS, but with RBTools 0.2
RC1, you could make use of the new --diff-filename. Basically, you'd
generate a diff using cvs diff and covering all revisions of that directory
only, and then pass the resulting filename using post-review
--diff-filename=<filename>

Christian

-- 
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com


On Mon, Feb 22, 2010 at 9:49 AM, <s...@pobox.com> wrote:

> I'm still getting familiar with Review Boar.  I use it like so:
>
>     * make a branch
>     * edit/debug/lather/rinse/repeat
>     * post-review
>     * visit url & fill in description, title, assign reviewers
>
> One of my colleagues asked a few of us to review a couple small
> applications
> written by a new hire.  This stuff had already been checked into our CVS
> repository.  I suspect this is not standard practice, but is there some way
> to post a review request for an entire application or an entire directory
> (that is, where there are no changes against the repository for the current
> directory)?  I see nothing obvious in post-review --help output which
> suggests this is possible.  I'm using post-review 0.8 and ReviewBoard 0.9
> ATM, but I have the power to upgrade it if necessary (it's just running on
> my desktop machine).
>
> Outside of me and a few other developers ReviewBoard is unknown at work, so
> I imagine this sort of "here, the application is finished, please review it
> all" request is going to pop up for some time to come.  I'd really like to
> be able to accommodate such requests as a way to demonstrate RB's
> usefulness.
>
> Thanks,
>
> --
> Skip Montanaro - s...@pobox.com - http://www.smontanaro.net/
>
> --
> Want to help the Review Board project? Donate today at
> http://www.reviewboard.org/donate/
> Happy user? Let us know at http://www.reviewboard.org/users/
> -~----------~----~----~----~------~----~------~--~---
> To unsubscribe from this group, send email to
> reviewboard+unsubscr...@googlegroups.com<reviewboard%2bunsubscr...@googlegroups.com>
> For more options, visit this group at
> http://groups.google.com/group/reviewboard?hl=en

-- 
Want to help the Review Board project? Donate today at 
http://www.reviewboard.org/donate/
Happy user? Let us know at http://www.reviewboard.org/users/
-~----------~----~----~----~------~----~------~--~---
To unsubscribe from this group, send email to 
reviewboard+unsubscr...@googlegroups.com
For more options, visit this group at 
http://groups.google.com/group/reviewboard?hl=en

Reply via email to