[PATCH] D69056: [LNT] Remove Python2 support

Thomas Preud'homme via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 21 03:17:54 PDT 2021


thopre added a comment.

In D69056#3064888 <https://reviews.llvm.org/D69056#3064888>, @thopre wrote:

> In D69056#2981941 <https://reviews.llvm.org/D69056#2981941>, @tnfchris wrote:
>
>> In D69056#2981915 <https://reviews.llvm.org/D69056#2981915>, @thopre wrote:
>>
>>> In D69056#2981910 <https://reviews.llvm.org/D69056#2981910>, @tnfchris wrote:
>>>
>>>> Thanks! this looks complete now.
>>>>
>>>> I guess the only question is, should we bump up the version number? to make a clear line in the sand? otherwise it's hard to tell people which version supports python 2 (in case they need to run an older one for whatever reason)? or more importantly, tell which one is python 3.
>>>
>>> Good point. We are currently in 0.4.2dev0 so perhaps we should tag a 0.4.2 first and then pass to 0.4.3dev0?
>>
>> Perhaps `0.5` instead? that would give someone wanting to continue Python 2 support for whatever reason a way to continue in the `0.4.x` ranges?
>
> Good point, release 0.4.2 and then change version to 0.5.0dev0 on the main branch. If we realize we want to do more releases with Python2 we can create a python2 branch from the release but I think we should not do it preemptively. Thought @cmatthews ?

Ping @cmatthews ?


Repository:
  rLNT LNT

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

https://reviews.llvm.org/D69056



More information about the llvm-commits mailing list