[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