<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On Oct 1, 2014, at 2:46 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com" class="">chandlerc@google.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class=""><div dir="ltr" class=""><div class="gmail_extra"><br class=""><div class="gmail_quote">On Wed, Oct 1, 2014 at 2:40 PM, Enrico Granata <span dir="ltr" class=""><<a href="mailto:egranata@apple.com" target="_blank" class="">egranata@apple.com</a>></span> wrote:<br class=""><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class=""><blockquote type="cite" class=""><div class="">On Oct 1, 2014, at 2:36 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com" target="_blank" class="">chandlerc@google.com</a>> wrote:</div><br class=""><div class=""><div dir="ltr" class="">This broke the CMake build and all the bots using it. Are you working on a fix?</div></div></blockquote><div class=""><br class=""></div></span><div class="">I didn’t receive any notification that the bots are broken - so I honestly can’t say I am working on a fix for a problem I am not aware of</div><div class="">I am aware of it now that you mention it, of course, and yes we should fix it</div></blockquote></div><br class="">omg, wow, we don't even have a build bot covering cmake with LLDB.</div><div class="gmail_extra"><br class=""></div><div class="gmail_extra">So, the deal is that LLVM has two supported build systems for *all* of its projects: configure+make and cmake. You have to update cmake when adding a new file. We have build bots for every project *except* LLDB using cmake to help catch when this doesn't happen... I have no idea why we don't have a build bot covering LLDB+CMake. That has to be fixed immediately.</div><div class="gmail_extra"><br class=""></div></div></div></blockquote><div><br class=""></div><div>We mostly use Xcode for builds at Apple, so it is really easy to remember to add files to the .xcodeproj but not as easy to remember about CMake. Not an excuse for why we (me) get to be bad at it, just an explanation</div><div>Bots would be nice, both as a reminder in scenarios as simple as this, and generally as a more actionable description of the problem than “hey this build mode that you don’t use is broken"</div><br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_extra">So you understand why I think this is so serious -- even if I'm not actively working on LLDB, if I just have it checked out into my LLVM tree, the entire tree stops building if something LLDB breaks CMake.</div></div></div></blockquote><div><br class=""></div><div>At that point you could of course revert your local LLDB checkout if you’re not actively working on it (as a stopgap measure at least), but yes warn-then-revert is also a fair response mode.</div><br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_extra"> =[ Anyways, while it would be great of LLDB devs actually tested the CMake build, the critical thing is to get a build bot covering it.</div><div class="gmail_extra"><br class=""></div><div class="gmail_extra">I've fixed this for now in r218831 so I can make progress again</div></div></div></blockquote><div><br class=""></div><div>Thanks! And sorry for the breakage of course!</div><br class=""><blockquote type="cite" class=""><div class=""><div dir="ltr" class=""><div class="gmail_extra">I've also asked Zach, Reid, or David to work on setting up a build bot.</div></div>
</div></blockquote></div><div><br class=""></div><br class=""><div class="">
<div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div style="color: rgb(0, 0, 0); letter-spacing: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><div class="">Thanks,</div><div class=""><i class="">- Enrico</i><br class="">📩 egranata@<font color="#ff2600" class=""></font>.com ☎️ 27683</div><div class=""><br class=""></div></div></div></div></div></div><br class="Apple-interchange-newline"><br class="Apple-interchange-newline">
</div>
<br class=""></body></html>