<div dir="ltr">On Wed, Aug 21, 2013 at 1:25 AM, Ahmed Bougacha <span dir="ltr"><<a href="mailto:ahmed.bougacha@gmail.com" target="_blank" class="cremed">ahmed.bougacha@gmail.com</a>></span> wrote:<br><div class="gmail_extra">
<div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div class="im">On Wed, Aug 21, 2013 at 1:19 AM, Chandler Carruth <<a href="mailto:chandlerc@google.com" class="cremed">chandlerc@google.com</a>> wrote:<br>

><br>
> On Wed, Aug 21, 2013 at 1:18 AM, Chandler Carruth <<a href="mailto:chandlerc@google.com" class="cremed">chandlerc@google.com</a>><br>
> wrote:<br>
>><br>
>> econdly, it makes build bots useless at tell you which commit broke<br>
>> something and allowing fine grained rollbacks.<br>
><br>
><br>
> Notably, one of your patches broke a bunch of build bots.<br>
<br>
</div>Hmm, I'm missing this one, link please?<br></blockquote><div><br></div><div><a href="http://bb.pgr.jp/builders/ninja-clang-i686-msc17-R/builds/4129">http://bb.pgr.jp/builders/ninja-clang-i686-msc17-R/builds/4129</a><br>
</div><div><br></div><div>Of course, half the patches didn't fit into that *huge* list for this build, so there is another build already queued up. It will take some time for it to finish and validate that a later commit fixed the prior commit.</div>
<div><br></div><div>However, I didn't even see a mention in a commit that one fixes a build break in the other, so all of this is really confusing to anyone working while you pushed that ends up checked out half way through the stream.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<div class="im"><br>
> Please fix these<br>
> *very quickly* or I will roll back the entire lot of them because it is just<br>
> too disruptive to do this to other developers. Please don't ever commit like<br>
> this again. =/<br>
<br>
</div>Terribly sorry, more in a bit.<br>
<span class=""><font color="#888888"><br>
-- Ahmed<br>
</font></span></blockquote></div><br></div></div>