<div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Wed, Sep 24, 2014 at 1:04 PM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":6l1" class="a3s" style="overflow:hidden">Hi Kaelyn,<br>
<br>
Please don't revert patches just because it breaks one buildbot,<br>
especially if the fix is obvious and only in the test. This makes it<br>
hard to re-apply them later, and creates a lot of work for re-testing<br>
on slow hardware.<br></div></blockquote><div><br></div><div>I specifically disagree. The policy for LLVM is to revert-to-green.</div><div><br></div><div>If a patch breaks a build bot, it is totally within scope to revert. If it isn't blocking you making progress, then not a big deal to leave it in and try to let the author fix it up.</div><div><br></div><div>But if the broken build bot represents your inability to build / test / etc in any way, reverting to green is totally the right response.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div id=":6l1" class="a3s" style="overflow:hidden">
<br>
I saw the email, I was fixing it, but when you reverted, I had to<br>
start all over again. Next time, check the email, IRC or just mail the<br>
person responsible, and let them fix it.</div></blockquote></div><br></div><div class="gmail_extra">She did email about this.</div><div class="gmail_extra"><br></div><div class="gmail_extra">I'm not sure why you had to start all over again? My version control makes it super easy to stash a fix, sync, un-revert, and re-apply my fix?</div><div class="gmail_extra"><br></div><div class="gmail_extra">This is a reasonably common occurrence in LLVM for me and hasn't caused any problems.</div></div>