<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sun, Sep 13, 2020 at 6:37 AM Paul C. Anagnostopoulos via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Yes, you are correct. The second commit should have been an amend. I shall now endeavor to check each commit before I push it, to ensure there is exactly one.<br>
<br>
My concern was how to restart the build with the second commit, but apparently that is automatic or someone did it for me. Thanks to all for your patience!<br></blockquote><div><br>Yeah, it's automatic - the bots keep rolling new revisions without need for manual intervention. If you commit a breakage the best you can do is commit the fix (if the forward fix is uncertain/complex, may be better to commit a revert as a fix) as soon as possible (while being as careful as possible not to introduce other issues) to reduce the window of breakage. (if you've already committed the fix, as in your case, then there's nothing to do - watch the bots, see that they go green again after picking up that change, etc)<br><br>- Dave<br> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
<br>
<br>
At 9/13/2020 09:24 AM, Hubert Tong wrote:<br>
>On Sun, Sep 13, 2020 at 8:12 AM Paul C. Anagnostopoulos via llvm-dev <<mailto:<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>><a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br>
>At 9/12/2020 09:45 PM, Mehdi AMINI wrote:<br>
>>I'm on Windows. Is there a way to set up that hook?<br>
>><br>
>>Ah sorry I'm not familiar enough with Windows, maybe someone else can chime in (and then we could update the doc as well).<br>
><br>
>I'm standing by waiting to learn what to do about the broken build . . .<br>
><br>
>The build was not (at the time when I looked yesterday) broken, so I was too confused to say more then.<br>
><br>
>I think I understand the situation now: You did not intend to push either e8e3693ceaa1afe267f21d2ba8d9565ea8fe7c12 or 93b4f8538267e620de4a36e7cf0abc0d4f8d7c10, but the squashed version of the two. If that's the case, there's nothing to do now: the first left the tree in a bad state and one after fixed it.<br>
<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div></div>