On Tue, Dec 11, 2012 at 12:17 PM, Emil A Eklund <e...@chromium.org> wrote:

> If the build is broken and the gardener/build cop has a strong reason
> to suspect that it was caused by a specific patch and the author is
> unavailable then rolling that patch out is the right thing to do.


Sure. If the author isn't available via IRC & emails within a reasonable
time, and the regression is as serious as a build failure, then rolling out
the patch is quite reasonable even if it's speculative.

On the other hand, if the patch being rolled out turned out be not the
cause of whatever failure the person rolled it out for, then it should be
his/her responsibility to re-land the patch.

It might inconvenience the author but it is the responsibility of the
> author and reviewer to make sure the patch didn't break anything.
>

Sure.

- R. Niwa
_______________________________________________
webkit-dev mailing list
webkit-dev@lists.webkit.org
http://lists.webkit.org/mailman/listinfo/webkit-dev

Reply via email to