[PATCH] D71565: Make mangled_names.test and update_cc_test_checks.py work with Python 2.

Mon Dec 16 21:41:07 PST 2019

MaskRay added a comment.

In D71565#1787219 <https://reviews.llvm.org/D71565#1787219>, @thakis wrote:

> In D71565#1786966 <https://reviews.llvm.org/D71565#1786966>, @MaskRay wrote:
>
> > Do you have a platform where installing Python 3 instead of Python 2 will be such an insurmountable task? Python 2.7 will retire in 15 days.
>
>
> If llvm updates its project policy to say that only py 3.x is supported (which x?), I'm fine with that. For now, we still support 2.7, so this script too must stick to that.
>
> The "will retire in 15 days" is fairly meaningless, it's not like it'll just disappear then.

My understanding is that this does not interact with the infrastructure. `python3` should be pervasive enough that we probably don't have to worry that not enough build bots run `REQUIRES: python3`. The infrastructure itself may require Python 2.7, but that is different.

About whether we should add Python 2 compatibility to new features: http://lists.llvm.org/pipermail/llvm-dev/2019-December/137634.html

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

https://reviews.llvm.org/D71565