<div dir="ltr">Just a drive by comment since I don't have a vested interest in lldb-server or this CL at the moment.  But adding lldb-server as a dependency of lldb seems strange to me.  I mean I get it, it just seems strange.  They're not really dependencies in the normal sense.  If you just use "ninja" instead of "ninja lldb" you should get both targets, while still allowing "ninja lldb" to only build lldb, as it currently does.  <div><br></div><div>Up to you guys though</div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Sep 16, 2015 at 7:48 AM Todd Fiala via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org">lldb-commits@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">tfiala added a comment.<br>
<br>
> Okay I'll make an adjustment to turn it on for Darwin and setting it based on a flag.<br>
<br>
<br>
I'm going to do it slightly differently --- I'll set a flag in the modules config that indicates if lldb *wants* lldb-server.  The logic on whether to build it or not should, as before, be driven by the targets asked for.  So we're not really setting whether we build it or not, we're setting whether we want it if we're already building lldb.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D12899" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12899</a><br>
<br>
<br>
<br>
_______________________________________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br>
</blockquote></div>