Re: [PATCH v1 1/2] perf python: Remove python 2 scripting support
From: Namhyung Kim
Date: Mon Sep 30 2024 - 17:30:09 EST
On Thu, Sep 19, 2024 at 06:45:23AM +0200, Ian Rogers wrote:
> On Thu, Sep 19, 2024 at 3:29 AM Andi Kleen <ak@xxxxxxxxxxxxxxx> wrote:
> >
> > On Thu, Sep 19, 2024 at 12:54:17AM +0200, Ian Rogers wrote:
> > > Python2 was deprecated 4 years ago, remove support and workarounds.
> >
> > Nacked-by: Andi Kleen
> >
> > I don't see any advantages of breaking perfectly fine existing setups
> > for no benefits.
Well, I think the benefit is in the maintenance. The EOL of Python 2
was 2020/1/1 [1] and we are targeting this release (v6.13) in 2025. So
I think it's reasonable to consider removing Python 2 support now.
>
> The reason is that since moving to linking libraries rather than
> building code we now have the ability to call functions like
> parse_events - previously parse_events was stubbed out because there
> was no way to build lex/yacc code. Calling parse_events is more
> sensible than existing logic that does things like open a legacy
> cycles event only on 1 PMU type. That is, the existing code doesn't
> support hybrid. Practically there is no way to test python2 support
> builds and works without having python2. We shouldn't be in the
> business of installing python2 in 2024, my organization has removed
> all support for over 3 years.
I think the problem nowdays is the container images for old distro. I'm
not sure how long we need to support them but it should be cut out at
some point.
Thanks,
Namhyung
[1] https://www.python.org/doc/sunset-python-2/