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.
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. On Sat, Oct 3, 2015 at 6:30 PM Todd Fiala via lldb-commits < lldb-commits@lists.llvm.org> wrote: > Author: tfiala > Date: Sat Oct 3 20:28:51 2015 > New Revision: 249256 > > URL: http://llvm.org/viewvc/llvm-project?rev=249256&view=rev > Log: > cmake: ensure readline python module target is added before finishing swig. > > When the readline target exists (only for non-Android Linux currently), > ensure that target is made a dependency of the finish_swig python-wrap-up > steps. This ensures it is built when building the lldb target. > > Fixes: > https://llvm.org/bugs/show_bug.cgi?id=25038 > > Modified: > lldb/trunk/CMakeLists.txt > > Modified: lldb/trunk/CMakeLists.txt > URL: > http://llvm.org/viewvc/llvm-project/lldb/trunk/CMakeLists.txt?rev=249256&r1=249255&r2=249256&view=diff > > ============================================================================== > --- lldb/trunk/CMakeLists.txt (original) > +++ lldb/trunk/CMakeLists.txt Sat Oct 3 20:28:51 2015 > @@ -24,6 +24,12 @@ if (NOT LLDB_DISABLE_PYTHON) > # We depend on liblldb being built before we can do this step. > add_dependencies(finish_swig liblldb argdumper) > > + # If we build the readline module, we depend on that happening > + # first. > + if (TARGET readline) > + add_dependencies(finish_swig readline) > + endif() > + > # Ensure we do the python post-build step when building lldb. > add_dependencies(lldb finish_swig) > > > > _______________________________________________ > lldb-commits mailing list > lldb-commits@lists.llvm.org > http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits >
_______________________________________________ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
