[Lldb-commits] [PATCH] D88840: [dotest] Simplify logic to find the Python path

Tue Oct 6 11:07:27 PDT 2020

labath added a comment.

In D88840#2314860 <https://reviews.llvm.org/D88840#2314860>, @JDevlieghere wrote:

> In D88840#2314594 <https://reviews.llvm.org/D88840#2314594>, @labath wrote:
>
>> One way this could be simplified further is to ditch `-P` and pass down the appropriate value from cmake/lit.
>
> LLDB in Xcode supports both Python 2 and 3 and we use a default `com.apple.dt.lldb DefaultPythonVersion` to override the version. By using `-P` here we can run the test suite with both versions by just changing the value of the default without having to reconfigure CMake. I'm not saying we should not do this because our donwstream fork, just that it would complicate things for us.

Yea, I'm not volunteering to do that (though I might one day), but... doesn't the python version that lldb uses and the python version running dotest needs to match? I'd expect that you'd still need to change /something/ so that it runs the different version of the python interpreter...

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D88840