public inbox for openembedded-core@lists.openembedded.org
 help / color / mirror / Atom feed
From: Richard Purdie <richard.purdie@linuxfoundation.org>
To: Jiaying.Song.CN@windriver.com, openembedded-core@lists.openembedded.org
Cc: alex.kanavin@gmail.com
Subject: Re: [OE-core] [scarthgap][master][PATCH] python3: provide /usr/bin/python as a symlink to python3
Date: Wed, 24 Jul 2024 17:01:23 +0100	[thread overview]
Message-ID: <78ec945a12ed570b274041fe8b14568e270fab06.camel@linuxfoundation.org> (raw)
In-Reply-To: <20240717023201.861394-1-jiaying.song.cn@windriver.com>

On Wed, 2024-07-17 at 10:32 +0800, Song, Jiaying (CN) via lists.openembedded.org wrote:
> From: Jiaying Song <jiaying.song.cn@windriver.com>
> 
> The /usr/bin/python cannot find the correct Python interpreter, so a symbolic
> link is added to point it to the Python3 interpreter.
> 
> Signed-off-by: Jiaying Song <jiaying.song.cn@windriver.com>
> ---
>  meta/recipes-devtools/python/python3/python3-manifest.json | 1 +
>  meta/recipes-devtools/python/python3_3.12.4.bb             | 1 +
>  2 files changed, 2 insertions(+)

There is a bit of a debate around this change. 

We're at a point where we patch anything using "python" to use python3.
This puts us in a fairly good deterministic position where everything
is explicit.

If we add this patch, on the plus side, we can drop some of the patches
we're carrying but we also lose the markup of the python versions. If
there ever was a python4, we'd be back in a mess again.

Adding it, starts discussions about whether it should just be present
unconditionally or whether it should be a configurable link with
update-alternatives. Should it be a separate recipe? Should any recipe
using "python" without a version have a specific dependency on this new
recipe/package?

Having listened to the discussions and given this is pretty much a
solved problem for the majority of our current metadata, I'm in favour
of maintaining the status quo and not taking this change, even if we
have to carry a few patches. It does at least make the situation quite
clear and explicit.

Cheers,

Richard


  parent reply	other threads:[~2024-07-24 16:01 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-07-17  2:32 [scarthgap][master][PATCH] python3: provide /usr/bin/python as a symlink to python3 jiaying.song.cn
2024-07-18 22:47 ` [OE-core] " Denys Dmytriyenko
2024-07-19  6:17   ` Alexander Kanavin
2024-07-19  6:18     ` Alexander Kanavin
2024-07-19 16:46       ` Denys Dmytriyenko
2024-07-23 13:38         ` Konrad Weihmann
2024-07-23 16:44           ` [OE-core] " Randy MacLeod
2024-07-23 17:56             ` Alexander Kanavin
2024-07-23 18:18               ` Konrad Weihmann
2024-07-23 18:25                 ` Alexander Kanavin
2024-07-23 18:41               ` Randy MacLeod
2024-07-24 16:01 ` Richard Purdie [this message]
2024-07-24 16:17   ` Alexander Kanavin
2024-07-24 16:36     ` Martin Jansa
2024-07-24 16:44       ` Alexander Kanavin

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=78ec945a12ed570b274041fe8b14568e270fab06.camel@linuxfoundation.org \
    --to=richard.purdie@linuxfoundation.org \
    --cc=Jiaying.Song.CN@windriver.com \
    --cc=alex.kanavin@gmail.com \
    --cc=openembedded-core@lists.openembedded.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox