<div dir="ltr">You just reminded me.  Since you've been hitting a lot of this stuff lately and doing a lot of great cleanup work, how do you feel about integrating the swig python scripts into the Xcode build?  I've been meaning to do this for a long time but I don't have enough Xcode knowledge.  Having parallel scripts for Xcode and CMake introduces an obvious technical debt that we could get rid of if there was just one script.<div><br></div><div>As far as I know it should be a drop-in replacement, and it supports everything the shell scripts currently support.  It's been lingering long enough that there's obviously no rush, but if you ever feel the urge to look at it, I think it would be easy.</div></div><br><div class="gmail_quote"><div dir="ltr">On Sat, Oct 3, 2015 at 6:30 PM 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">Author: tfiala<br>
Date: Sat Oct  3 20:28:51 2015<br>
New Revision: 249256<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=249256&view=rev" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project?rev=249256&view=rev</a><br>
Log:<br>
cmake: ensure readline python module target is added before finishing swig.<br>
<br>
When the readline target exists (only for non-Android Linux currently),<br>
ensure that target is made a dependency of the finish_swig python-wrap-up<br>
steps.  This ensures it is built when building the lldb target.<br>
<br>
Fixes:<br>
<a href="https://llvm.org/bugs/show_bug.cgi?id=25038" rel="noreferrer" target="_blank">https://llvm.org/bugs/show_bug.cgi?id=25038</a><br>
<br>
Modified:<br>
    lldb/trunk/CMakeLists.txt<br>
<br>
Modified: lldb/trunk/CMakeLists.txt<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/lldb/trunk/CMakeLists.txt?rev=249256&r1=249255&r2=249256&view=diff" rel="noreferrer" target="_blank">http://llvm.org/viewvc/llvm-project/lldb/trunk/CMakeLists.txt?rev=249256&r1=249255&r2=249256&view=diff</a><br>
==============================================================================<br>
--- lldb/trunk/CMakeLists.txt (original)<br>
+++ lldb/trunk/CMakeLists.txt Sat Oct  3 20:28:51 2015<br>
@@ -24,6 +24,12 @@ if (NOT LLDB_DISABLE_PYTHON)<br>
     # We depend on liblldb being built before we can do this step.<br>
     add_dependencies(finish_swig liblldb argdumper)<br>
<br>
+    # If we build the readline module, we depend on that happening<br>
+    # first.<br>
+    if (TARGET readline)<br>
+        add_dependencies(finish_swig readline)<br>
+    endif()<br>
+<br>
     # Ensure we do the python post-build step when building lldb.<br>
     add_dependencies(lldb finish_swig)<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>