clayborg added a comment.

In D61090#1477709 <https://reviews.llvm.org/D61090#1477709>, @davide wrote:

> @clayborg I would also like to point out that some of us fair amount of time 
> cleaning `LLDB_DISABLE_PYTHON` leaking all over the codebase, and this is 
> basically the last occurrence, so we're  motivated to have this going 
> away/fixed in some way.


Fine to have a ScriptInterpreterPython that is gutted if disabled. Or call some 
code that can dynamically detect if python is enabled and do the right thing 
inside "DefaultScriptInterpreter::GetLLDBModuleDirectory()"


Repository:
  rLLDB LLDB

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61090/new/

https://reviews.llvm.org/D61090



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to