[Lldb-commits] [PATCH] D29405: Install six.py conditionally

2017-02-03 Thread Pavel Labath via Phabricator via lldb-commits
labath accepted this revision. labath added a comment. This revision is now accepted and ready to land. looks good, thank you. Repository: rL LLVM https://reviews.llvm.org/D29405 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://li

[Lldb-commits] [PATCH] D29405: Install six.py conditionally

2017-02-03 Thread Kamil Rytarowski via Phabricator via lldb-commits
krytarowski updated this revision to Diff 87000. krytarowski edited the summary of this revision. krytarowski added a comment. Pass option to `--useSystemSix` to `finishSwigWrapperClasses.py` Repository: rL LLVM https://reviews.llvm.org/D29405 Files: CMakeLists.txt cmake/modules/LLDBConf

[Lldb-commits] [PATCH] D29405: Install six.py conditionally

2017-02-02 Thread Pavel Labath via Phabricator via lldb-commits
labath added a comment. Look at top level CmakeLists.txt file you will find an add_custom_target rule that runs it. Repository: rL LLVM https://reviews.llvm.org/D29405 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.or

[Lldb-commits] [PATCH] D29405: Install six.py conditionally

2017-02-02 Thread Kamil Rytarowski via Phabricator via lldb-commits
krytarowski added a comment. I will try to reverse engineer it.. Repository: rL LLVM https://reviews.llvm.org/D29405 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

[Lldb-commits] [PATCH] D29405: Install six.py conditionally

2017-02-02 Thread Kamil Rytarowski via Phabricator via lldb-commits
krytarowski added a comment. In https://reviews.llvm.org/D29405#664957, @labath wrote: > Unfortunately, this prevents six.py being copied into the build directory, so > a non-installed lldb will still not work (if you don't have the system > six.py). > > It looks like you will have to pass this

[Lldb-commits] [PATCH] D29405: Install six.py conditionally

2017-02-02 Thread Pavel Labath via Phabricator via lldb-commits
labath added a comment. Unfortunately, this prevents six.py being copied into the build directory, so a non-installed lldb will still not work (if you don't have the system six.py). It looks like you will have to pass this flag into finishSwigPythonLLDB.py and check the condition there. Repos

[Lldb-commits] [PATCH] D29405: Install six.py conditionally

2017-02-02 Thread Kamil Rytarowski via Phabricator via lldb-commits
krytarowski added a comment. @labath patch done. I'm specifying `-DLLDB_USE_SYSTEM_SIX:BOOL=TRUE` in pkgsrc. It works for me. Repository: rL LLVM https://reviews.llvm.org/D29405 ___ lldb-commits mailing list lldb-commits@lists.llvm.org http://l