linux-kernel - Re: [PATCH] kbuild: remove PYTHON variable

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [day] [month] [year] [list]

Message-ID: <87czxjv5zn.fsf@meer.lwn.net>
Date:   Mon, 01 Feb 2021 14:21:48 -0700
From:   Jonathan Corbet <corbet@....net>
To:     Masahiro Yamada <masahiroy@...nel.org>
Cc:     Linux Kbuild mailing list <linux-kbuild@...r.kernel.org>,
        "open list:DOCUMENTATION" <linux-doc@...r.kernel.org>,
        Fenghua Yu <fenghua.yu@...el.com>,
        Michal Marek <michal.lkml@...kovi.net>,
        Tony Luck <tony.luck@...el.com>, linux-ia64@...r.kernel.org,
        Linux Kernel Mailing List <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH] kbuild: remove PYTHON variable

Masahiro Yamada <masahiroy@...nel.org> writes:

> On Tue, Feb 2, 2021 at 1:38 AM Jonathan Corbet <corbet@....net> wrote:
>> So this will have the effect of making the docs build impossible for
>> folks who only have Python 2.
>
> Is this a problem?  Python 2 is EOL.
>
> Everybody who wants to use Python
> must install Python 3.

I honestly don't know, but people do keep things around for a
surprisingly long time.

>>  As I said before, this is a step that we
>> knew was coming, we just hadn't decided on the exact timing - I guess
>> this decides for us :)
>>
>> That said, I'll copy linux-doc to see if anybody screams.  I assume this
>> is a 5.12 change?
>
> No, I am planning to do this for 5.11
> (a pull request this week if there is no objection).

I'm not going to truly object and try to block this, but I worry that
it's soon and without warning.  It's not a change I would merge for
-rc7.  IMO this should be done in 5.12 with an appropriate note added to
Documentation/process/changes.rst as well.

Thanks,

jon