On Tue, Dec 11, 2012 at 1:11 PM, Emil A Eklund <e...@chromium.org> wrote:
> On Tue, Dec 11, 2012 at 12:19 PM, Ojan Vafai <o...@chromium.org> wrote: > > On Tue, Dec 11, 2012 at 12:17 PM, Emil A Eklund <e...@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. > > Indeed. > > > That said, if your strong reason turned out to be incorrect, you should > recommit the patch, no? > > That seems like a bad idea, someone that understands the patch should > recommit it. Ideally the original author. > If it needs manual patching then you need to include the original author, but otherwise, I don't see why.
_______________________________________________ webkit-dev mailing list webkit-dev@lists.webkit.org http://lists.webkit.org/mailman/listinfo/webkit-dev