[Lldb-commits] [PATCH] D112212: [lldb/test] Print build commands in trace mode

Fri Oct 22 16:09:15 PDT 2021

JDevlieghere added a comment.

In D112212#3080491 <https://reviews.llvm.org/D112212#3080491>, @teemperor wrote:

> This LGTM, but `shlex.join` is actually Py3 exclusive and I don't think there is a good Py2 replacement. I think we're just in time for the Py2->3 migration according to the timeline Jonas posted last year <https://lists.llvm.org/pipermail/lldb-dev/2020-August/016388.html>, so let's use this patch to actually do that? Then we can also get rid of all the `six` stuff etc.
>
> Let's see if Jonas has any objections against dropping Py2 with this, otherwise this is good to go.

We're planning to branch from open source on October 26th. If there's no urgency, it would really be great if we can hold off breaking Py2 until then.

I'm all in favor for getting rid of Python 2 support, but sweeping changes like dropping the `six` stuff will introduce a lot of headaches (merge conflicts) for us. If we could postpone that for another release that would save us a bunch of engineering time.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112212