[PATCH] D95635: [CMake] Actually require python 3.6 or greater

[Christopher Tetreault via Phabricator via llvm-commits]

ctetreau added a comment.

In D95635#2528851 <https://reviews.llvm.org/D95635#2528851>, @JDevlieghere wrote:

>> However, the project claims to require 3.6 or greater, and 3.6 features are being used.
>
> Can you link to where it says that? I'm not opposed to making 3.6 the minimally required version, but at least for LLDB we don't have that requirement and we have documentation mentioning Python 3.5.



In D95635#2528883 <https://reviews.llvm.org/D95635#2528883>, @yln wrote:

> In D95635#2528851 <https://reviews.llvm.org/D95635#2528851>, @JDevlieghere wrote:
>
>>> However, the project claims to require 3.6 or greater, and 3.6 features are being used.
>>
>> Can you link to where it says that? I'm not opposed to making 3.6 the minimally required version, but at least for LLDB we don't have that requirement and we have documentation mentioning Python 3.5.
>
> For LLVM: https://llvm.org/docs/GettingStarted.html#software

This is where I saw it. I got bit by @yln's cleanup patch. I can reduce the minimum version for LLDB if that's the officially required version, but llvm at the very least advertises 3.6


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95635