<div dir="ltr">Oh I see what you mean.<div><br></div><div>Yeah that would be easy enough to do. I'll put it on a cleanup-priority list. (Not promising I'll hit it very soon but I will do it).</div></div><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Oct 5, 2015 at 1:08 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"><div dir="ltr">So on the Xcode side, there is a script that runs called `build-swig-wrapper-classes.sh` that converts the SWIG into python stuff. On the Windows side, there's no such thing as shell scripts, so we wrote python scripts instead. It's called buildSwigWrapperClasses.py, in the same folder ad the shell script.<div><br></div><div>The command line syntax is different, but functionally they should be equivalent. To see what the syntax looks like for the python file, you could probably just edit the CMakeLists.txt and print out the command line during CMake generation time. Then change the shell script in the Xcode project to use the python script instead, and finally delete the shell script from the repo.</div></div><div class="HOEnZb"><div class="h5"><br><div class="gmail_quote"><div dir="ltr">On Mon, Oct 5, 2015 at 12:55 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"><div dir="ltr">Oops, missed this. I see how it can be gnarly finding things in lldb-commits!<div><br></div><div><br></div><div class="gmail_extra"><br><div class="gmail_quote"></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Sat, Oct 3, 2015 at 7:25 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"><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?</div></blockquote><div><br></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Can you tell me more about what you had in mind? (I am actually way more familiar with how we do it on the cmake side than the Xcode side!)</div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"> I've been meaning to do this for a long time but I don't have enough Xcode knowledge.</div></blockquote><div><br></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>I'm pretty sure I can do whatever we want to try to do here, as soon as I know what that is. The only thing that might be interesting is we have a group of external folks using a cmake build on OS X and not using xcodebuild. So whatever change you're thinking here needs to still work for a straight cmake approach (I think).</div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"> 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><div><div><br></div></div></blockquote><div><br></div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div>Sure, tell me a bit more and I can look into getting this on my queue.</div><div><br></div><div>-Todd</div></div></div></div><div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><div><br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div><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" target="_blank">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>
</div></div></blockquote></div></div></div><div dir="ltr"><div class="gmail_extra"><br><br clear="all"><div><br></div>-- <br><div><div dir="ltr">-Todd</div></div>
</div></div></blockquote></div>
</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div class="gmail_signature"><div dir="ltr">-Todd</div></div>
</div>