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



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

Reply via email to