Hi Gyula,

Thank you for the reply. I'm definitely interested in the syntax.  We
already rolled out the RB in our team, so this wasn't just an
exercise :).

Looking forward to hearing back from you in January

-- Dennis


On Dec 23, 4:15 pm, Gyula Faller <faller.gy...@gmail.com> wrote:
>  Hi,
>
>  You can send a revision range in case of Perforce too, but with a
> different syntax. You can specifiy a list of file ranges. I on holyday now,
> and I don't remember the exact form but something like
>
> post-review
> perforce-file-path1#file-start-revid,perforce-file-path1#file-stop-revid
> perforce-file-path2#file-start-revid,perforce-file-path2#file-stop-revid
>
> or you can use @chgnum instead of #revid as well.
>
> If you are interested in the exact syntax I can show you in January.
>
>   Gyula
>
>
>
>
>
>
>
> On Thu, Dec 22, 2011 at 9:47 PM, DXM <dxm...@gmail.com> wrote:
> > I know this question has been asked before (e.g. one of many:
>
> >http://groups.google.com/group/reviewboard/browse_thread/thread/41861...
> > ),
> > but it's still not clear if this functionality is actually supported.
>
> > I've read and reread the manual, specifically the paragraph on
> > reviewing committed code and more specifically on explanation of this
> > command: "$ post-review --revision-range=STARTREV:STOPREV"
>
> > I'm using Perforce.  Does RB support this with Perforce?  If it does,
> > what is "STARTREV" and "STOPREV"? are these file revisions or
> > changelist( aka changeset) numbers?  If they are changelist numbers,
> > is it going to include everyone else's changelists as well?  We have
> > about 200 people checking in code.  By the time I get around to
> > responding to a code review feedback, there can be a lot of other
> > changelists between original and the new one.  How does this work in
> > that scenario?  Is it possible to just provide a discrete list of
> > changelists that I want to have included in a review?
>
> > --
> > Want to help the Review Board project? Donate today at
> >http://www.reviewboard.org/donate/
> > Happy user? Let us know athttp://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

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