Re: Patch queue and reviewing (Was Re: Generator programs can only be built with optimization enabled?)

["Richard Guenther" <richard dot guenther at gmail dot com>]

On 6/15/06, Joe Buck <Joe.Buck@synopsys.com> wrote:
> On Thu, Jun 15, 2006 at 01:03:17PM -0400, Diego Novillo wrote:
> > Manuel López-Ibáñez wrote on 06/15/06 05:34:
> >
> > > I mean, there is a patch queue, you put your patch or patch
> > > set in the queue, it gets bootstrapped and tested as you said on 2-5
> > > patforms, then either it passes and a notification is send to the
> > >
> > Actually, the patch queue only stores the patches.  This idea of
> > bootstrapping on 2-5 platforms automatically has not been implemented.
>
> Right, but Manuel was commenting on Mike Stump's proposal, wondering
> why Mike proposed to run the bootstrap tests *after* reviewer approval
> instead of before.

It might be a question of resources, though I agree that it makes sense to do
it before patch review.  Like testing on all primary release platforms - which
makes resources more of a problem, as it is not simply throwing a bunch
of cheap boxes at the problem this way.

Richard.