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

[Michał Górny via Phabricator via lldb-commits]

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