
<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; "><br><div><div>On Oct 24, 2013, at 10:52 AM, Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">On Thu, Oct 24, 2013 at 10:19 AM, Argyrios Kyrtzidis <span dir="ltr"><<a href="mailto:kyrtzidis@apple.com" target="_blank" class="cremed">kyrtzidis@apple.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin: 0px 0px 0px 0.8ex; border-left-width: 1px; border-left-color: rgb(204, 204, 204); border-left-style: solid; padding-left: 1ex; position: static; z-index: auto; "><div>It looks like the failure was in clang-tools-extra (<a href="http://clang.llvm.org/docs/ClangTools.html" target="_blank" class="cremed">http://clang.llvm.org/docs/ClangTools.html</a>).</div>

<div>IMHO, since this is not in the clang repository, it is unreasonable to have the phase1 bots tied to that and expect everyone to build and update it after making changes.</div><div><br></div><div>Michael, what do you think ? I think we <i>should</i> have a bot building clang-tools-extra, but I don't think it belongs in phase 1.</div>

</blockquote></div><br></div><div class="gmail_extra">I don't care very much whether its in phase 1, but I think it is really, really important that developers making an API change are expected to update code in that tree.</div>

<div class="gmail_extra"><br></div><div class="gmail_extra">When changing an LLVM API, we routinely ask people to update Clang, DragonEgg, etc. This shouldn't be any different.</div></div>

</blockquote></div><div><br></div><div>Asking people to update clang-tools-extra is fine, that's why I believe there should be a buildbot building it, but phase 1 is the highest priority, any breaking window should be at a minimum; phase 1 should build the code that everyone committing changes is building, not include repositories that are out of the core repositories.</div><br></body></html>