Re: [PATCH v4 1/2] init/Kconfig: add python3 availability config

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Wed, Nov 20, 2024 at 5:17 PM Petr Vorel <pvorel@xxxxxxx> wrote:
>
> > On Thu, Nov 21, 2024 at 5:41 AM Petr Vorel <pvorel@xxxxxxx> wrote:
>
> > > It will be used in the next commit for DRM_MSM.
>
> > > Suggested-by: Rob Clark <robdclark@xxxxxxxxx>
> > > Signed-off-by: Petr Vorel <pvorel@xxxxxxx>
> > > ---
> > > Changes v3->v4:
> > > * Move definition to the end of the file
>
>
> > I prefer to not check the tool.
>
> Ack.
>
> > Why don't you install python3?
>
> Everybody installs it when it's required, the question is how to inform about
> the dependency.
>
> There build environments are minimal environments:
> * chroot (e.g. cross compilation)
> * container
>
> These are used by both developers and distros.

I don't think py3 is an _onerous_ dependency, but it has come up as a
surprise in minimal distro build environments at least once.. so I'd
be a fan of surfacing this dependency in a predictable/understandable
way (ie. I'm in favor of this patchset)

BR,
-R

> Kind regards,
> Petr
>
> > >  init/Kconfig | 3 +++
> > >  1 file changed, 3 insertions(+)
>
> > > diff --git a/init/Kconfig b/init/Kconfig
> > > index fbd0cb06a50a..c77e45484e81 100644
> > > --- a/init/Kconfig
> > > +++ b/init/Kconfig
> > > @@ -2047,3 +2047,6 @@ config ARCH_HAS_SYNC_CORE_BEFORE_USERMODE
> > >  # <asm/syscall_wrapper.h>.
> > >  config ARCH_HAS_SYSCALL_WRAPPER
> > >         def_bool n
> > > +
> > > +config HAVE_PYTHON3
> > > +       def_bool $(success,$(PYTHON3) -V)
> > > --
> > > 2.45.2





[Index of Archives]     [Linux&nblp;USB Development]     [Linux Media]     [Video for Linux]     [Linux Audio Users]     [Yosemite Secrets]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux