<div dir="ltr">I'm taking a look now. In the future please revert changes that break a buildbot.  If you think you might know what the fix is (i.e. you guess that including a header will fix it), then you can just check it in as a test.  If after a few attempts you still can't figure it out, then revert the change out again.<div><br></div><div>But definitely please don't leave broken changes in.  For example, the MSVC buildbot is now failing due to changes in LLVM.  The reason is because it broke last night due to some unrelated changes which weren't reverted, and subsequent breaks did not result in an email to other people so they didn't know.  We need to avoid this situation, so the bot has to stay green.</div><div><br></div><div>anyway I'm working on fixing it now</div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Oct 22, 2015 at 9:28 AM Eugene Zelenko via lldb-dev <<a href="mailto:lldb-dev@lists.llvm.org">lldb-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi!<br>
<br>
I broke couple of times MSVC builds when committed minor code<br>
cleanups. Looks like problem occurred because of changed order of<br>
headers.<br>
<br>
I could not investigated problems myself, because I don't have access to MSVC.<br>
<br>
Changes in question:<br>
<br>
r250872:<br>
<br>
source/Plugins/Disassembler/llvm/DisassemblerLLVMC.cpp<br>
source/Plugins/Disassembler/llvm/DisassemblerLLVMC.h<br>
<br>
r250925:<br>
<br>
source/Plugins/OperatingSystem/Go/OperatingSystemGo.cpp<br>
source/Plugins/OperatingSystem/Go/OperatingSystemGo.h<br>
<br>
Eugene.<br>
_______________________________________________<br>
lldb-dev mailing list<br>
<a href="mailto:lldb-dev@lists.llvm.org" target="_blank">lldb-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-dev</a><br>
</blockquote></div>