[Lldb-commits] [PATCH] D64881: [Cmake] Use the modern way to find Python when possible

Michał Górny via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Wed Jul 17 22:44:16 PDT 2019


mgorny added a comment.

That said, I ended up having mixed feelings about this.

On the plus side, it definitely forces matching Python version for me. For example, without this patch changing `PYTHON_EXECUTABLE` resulted in mismatch. If I wanted to change Python version, I had to either update all vars manually or remove them from cache.

On the minus side, I can't seem to find a way to force a particular interpreter. Overriding `Python_EXECUTABLE` just doesn't work, and it always forces 3.7 for me. The logic is rather complex, and I'm not sure if it's possible at all. Maybe we should wait for the code to mature a bit.


Repository:
  rLLDB LLDB

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

https://reviews.llvm.org/D64881





More information about the lldb-commits mailing list