<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Sep 19, 2016 at 7:39 PM, Zachary Turner <span dir="ltr"><<a href="mailto:zturner@google.com" target="_blank">zturner@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Unless someone has a strong desire to keep it and it adds necessary functionality I would vote for removing it.</blockquote><div><br></div><div>I think it's fine to remove the other support if nobody else has a good reason to want to keep it, but we're not to that point yet.  We are going to want to keep the switch on and default to the existing cmake macOS build until Chris completes the work here.  This change as is is not ready for prime time.  It's needs full-on verification against our existing Xcode build, and I think Chris mentioned he has other bits he needs to do to make it complete.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">  It can always be maintained downstream.  That being said, let's wait and see if anyone chimes in.<br><br>I also want to test this tomorrow to make sure nothing breaks.<br><br></blockquote><div><br></div><div>Yeah, this also needs checking on Linux.  I can do that tomorrow if Pavel doesn't get to it first.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Just to be clear, if i were to apply this on mac into a canonical llvm-style directory hierarchy and run ninja check-lldb, is that supposed to work?<br><div class="gmail_quote"><div dir="ltr">On Mon, Sep 19, 2016 at 6:36 PM Todd Fiala <<a href="mailto:todd.fiala@gmail.com" target="_blank">todd.fiala@gmail.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">tfiala accepted this revision.<br>
tfiala added a subscriber: labath.<br>
tfiala added a comment.<br>
This revision is now accepted and ready to land.<br>
<br>
LGTM.<br>
<br>
Chris - you might want to add @labath to this as well.<br>
<br>
<br>
================<br>
Comment at: source/API/CMakeLists.txt:9<br>
@@ -8,1 +8,3 @@<br>
<br>
+option(LLDB_BUILD_FRAMEWORK "Build the Darwin LLDB.framework" Off)<br>
+<br>
----------------<br>
zturner wrote:<br>
> Is there any reason to have this off when building on OSX?  Is there any value in having it not be an option at all, but just if you're on Darwin, you just always get a Framework?<br>
I mentioned to Chris that I wasn't sure if the Google folks (or maybe Dawn) that build with the CMake build might want to keep it building the old way.<br>
<br>
I'd be all for only building only the framework way with CMake if nobody else particularly cared.  That would simplify things.  But I don't want to force it if anybody else cares.<br>
<br>
Our goal is to get the CMake-based build producing the same exact thing as our Xcode-based build.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D24749" rel="noreferrer" target="_blank">https://reviews.llvm.org/<wbr>D24749</a><br>
<br>
<br>
<br>
</blockquote></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature" data-smartmail="gmail_signature"><div dir="ltr">-Todd</div></div>
</div></div>