Unless someone has a strong desire to keep it and it adds necessary functionality I would vote for removing it.  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>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">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 class="gmail_msg">
tfiala added a subscriber: labath.<br class="gmail_msg">
tfiala added a comment.<br class="gmail_msg">
This revision is now accepted and ready to land.<br class="gmail_msg">
<br class="gmail_msg">
LGTM.<br class="gmail_msg">
<br class="gmail_msg">
Chris - you might want to add @labath to this as well.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
================<br class="gmail_msg">
Comment at: source/API/CMakeLists.txt:9<br class="gmail_msg">
@@ -8,1 +8,3 @@<br class="gmail_msg">
<br class="gmail_msg">
+option(LLDB_BUILD_FRAMEWORK "Build the Darwin LLDB.framework" Off)<br class="gmail_msg">
+<br class="gmail_msg">
----------------<br class="gmail_msg">
zturner wrote:<br class="gmail_msg">
> 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 class="gmail_msg">
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 class="gmail_msg">
<br class="gmail_msg">
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 class="gmail_msg">
<br class="gmail_msg">
Our goal is to get the CMake-based build producing the same exact thing as our Xcode-based build.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D24749" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D24749</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div>