<div dir="ltr">LGTM, thanks!<div><br></div><div>--renato</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">On 25 November 2013 12:31, Mikael Lyngvig <span dir="ltr"><<a href="mailto:mikael@lyngvig.org" target="_blank">mikael@lyngvig.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Here's what I hope to be the final patch.  I've even proof-read it line by line (I always do a diff before committing, but obviously I must have been skimming too fast).<div>
<br></div><div>Yes, I can imagine what a nightmare it must have been.  Feverish coders en masse battling for supremacy with each their own ill thought through patches. </div>
<div><br></div><div>I apologize for my numerous errors this day, but I've been up too long, working too hard, so now the failure rate is increasing exponentially, it seems.  Time to shut down for today.</div><div><div class="h5">
<div class="gmail_extra">
<br><br><div class="gmail_quote">2013/11/25 Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On 25 November 2013 12:21, Mikael Lyngvig <span dir="ltr"><<a href="mailto:mikael@lyngvig.org" target="_blank">mikael@lyngvig.org</a>></span> wrote:<br>


<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">I did the unthinkable and edited the previous patch when I enabled the test so now that change got lost when I sent you the last patch.  Do you mind editing the "test=False" to become "test=True" in the patch I just sent you?</div>


</blockquote><div><br></div><div>Heh, I was just about to comment on that. ;)</div><div><br></div><div>You can send a final patch to the list with both changes in. I prefer if Galina commits the Zorg patches, since she knows when the buildmaster will restart.</div>


<div><br></div><div>The changes look good to me with 4 jobs and test=true.</div><div><br></div><div>cheers,</div><div>--renato</div><div><br></div><div>PS: I can't remember how it was like before peer review, but it surely was horrible!</div>


</div></div></div>
</blockquote></div><br></div></div></div></div>
</blockquote></div><br></div>