[PATCH] D59719: [ScriptInterpreter] Make sure that PYTHONHOME is right.

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 25 18:07:18 PDT 2019


davide added a comment.

In D59719#1441254 <https://reviews.llvm.org/D59719#1441254>, @labath wrote:

> It sounds to me like you could achieve the same thing by generalizing the LLDB_PYTHON_HOME logic in LLDBConfig.cmake. This would have the advantage of centralizing the way we manage python-finding logic (instead of each OS doing it's own thing) and also enable those users, who know what they are doing, to override this logic and point lldb to a different python. (I don't know if there are any such users, but it does not sounds like an impossible scenario).
>
> I think all it would take is to do something like:
>
> - move LLDB_RELOCATABLE_PYTHON handling outside of `if(WINDOWS)`
> - have the default value of `LLDB_RELOCATABLE_PYTHON` be `false` for darwin
> - possibly tweak the python-finding logic so that it prefers the one in /System/Library/Frameworks/...


I'll try to make this more generic. Thanks for the input.


Repository:
  rLLDB LLDB

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

https://reviews.llvm.org/D59719





More information about the llvm-commits mailing list