[asterisk-dev] [Code Review] Fix multiple parking issues.

[Shaun Ruffell]

On Thu, Aug 11, 2011 at 11:39:53AM -0500, Kevin P. Fleming wrote:
> On 08/11/2011 10:25 AM, Shaun Ruffell wrote:
> >On Thu, Aug 11, 2011 at 02:28:40PM -0000, jrose wrote:
> >>
> >>Honestly, patches this huge are very difficult to track with
> >>reviewboard, and in fact I don't really believe there is a diff viewer
> >>on Earth that could adequately portray something with this much
> >>reordering...
> >
> >Hopefully in the not-too-distant future we will have a good way to
> >handle patch series instead of individual patches. A side effect of
> >taking large groups of related, but independent, commits as patch series
> >is ease of bisecting for regressions after the fact.
> >
> >Looking at the description in https://reviewboard.asterisk.org/r/1358
> >leads me to believe this change is a prime candidate.
> 
> We already do, and we're testing it. Crucible can handle reviews
> from branches without collapsing all the revisions into a single
> diff (unless you want it to), and I believe it can also handle
> uploading multiple patches into a single review (although I don't
> think any of us have tested that).

Very cool. I guess it's time for me to join in the Crucible testing...

-- 
Shaun Ruffell
Digium, Inc. | Linux Kernel Developer
445 Jan Davis Drive NW - Huntsville, AL 35806 - USA
Check us out at: www.digium.com & www.asterisk.org