Yes, it should work with 1.0.5.x.
You should certainly be able to do the multiple patch thing today, though as
Dan said, not elegantly. I think down the road we're going to want to look
into patchset functionality. Internally, we already have a concept of a
revisioned set of diffs (a "DiffSetHistory"). Today, a review request points
to a single one of these, but in the future, we should be able to modify
this so that it can point to more than one. The diff viewer would then allow
switching between these.
This would be a great Summer of Code project, actually. It's probably not
all that difficult, though we'd need to figure out some
backwards-compatibility issues, and make the API work well with it.
I haven't given this much thought yet, but what I'm envisioning is that the
diff viewer, when linking to multiple patchsets, would show a list of each
one somewhere above the revision selector, and the user could specify which
one to look at. Each patchset would be capable of having multiple revisions
of diffs just like today (though I'm not entirely sure how useful this is --
maybe outside of the standard Git patchset model).
Christian Hammond - chip...@chipx86.com
Review Board - http://www.reviewboard.org
VMware, Inc. - http://www.vmware.com
On Thu, Feb 4, 2010 at 11:45 AM, Dan Savilonis <d...@n-cube.org> wrote:
> See the Distributed Version Control Systems section of:
> It's definitely available, and I believe it should work with git in
> 220.127.116.11, but TBH I've never run the stable version of RB.
> In a 3 patch series, for example, you'd submit the first patch
> directly, then the second patch with a parent diff of the first patch,
> then the third patch with a parent diff of the sum of the first two
> On Feb 4, 10:33 am, Stephen Gallagher <karrde...@gmail.com> wrote:
> > On Feb 4, 10:20 am, Dan Savilonis <d...@n-cube.org> wrote:
> > > post-review can actually generate any arbitrary diff, not just a diff
> > > against HEAD. Make sure you're using the latest nightly build of
> > > rbtools for this functionality. It also supports a 'parent diff' mode
> > > where you can specify a parent of your diff set such that if you are
> > > submitting a series of patches not available in the server's repo, RB
> > > will still be able to understand how to generate the side-by-side
> > > diff.
> > Can you point me at some documentation for the parent diff mode? I'd
> > like to understand more about this. Is it available in the stable
> > 18.104.22.168, or does it require the 1.1 alpha?
> Want to help the Review Board project? Donate today at
> Happy user? Let us know at http://www.reviewboard.org/users/
> To unsubscribe from this group, send email to
> For more options, visit this group at
Want to help the Review Board project? Donate today at
Happy user? Let us know at http://www.reviewboard.org/users/
To unsubscribe from this group, send email to
For more options, visit this group at