[webkit-dev] Please avoid rolling out patches speculatively and reland them ASAP if you had to

Ojan Vafai ojan at chromium.org
Tue Dec 11 12:19:52 PST 2012


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

> I'll have to disagree with you here.
>
> 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. It
>

"author is unavailable" is the key statement here. That said, if your
strong reason turned out to be incorrect, you should recommit the patch, no?


> might inconvenience the author but it is the responsibility of the
> author and reviewer to make sure the patch didn't break anything.
> _______________________________________________
> webkit-dev mailing list
> webkit-dev at lists.webkit.org
> http://lists.webkit.org/mailman/listinfo/webkit-dev
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.webkit.org/pipermail/webkit-dev/attachments/20121211/44c83a68/attachment.html>


More information about the webkit-dev mailing list