[PATCH 1/2] Allow Python packages to be dropped

[Simon Glass]

Hi Tom,

On Thu, 31 Aug 2023 at 11:48, Tom Rini <trini at konsulko.com> wrote:
>
> On Thu, Aug 31, 2023 at 11:20:52AM -0600, Simon Glass wrote:
>
> > When building in a portage chroot, we do not have the environment needed
> > to build pylibfdt. It is instead build as a separate package.
> >
> > Provide a build option to tell U-Boot to skip this part of the build. We
> > still need it to use binman, etc. but don't need it to build its
> > dependencies.
> >
> > Signed-off-by: Simon Glass <sjg at chromium.org>
> > Reviewed-by: Mike Frysinger <vapier at chromium.org>
> > ---
> >
> >  Makefile             | 8 ++++++++
>
> Can we do this via Kconfig instead? It looks like we don't need to do
> this part of the build for coreboot, yes?  And so if we can drop it from
> other builds as well that might help speed up CI.

Yes it could. But I really don't like changing the board config on the
fly - we have things like 'select BINMAN' now and it just gets messy.
In the normal course of events we want to build the python things and
the build won't work without it (e.g. binman will fail). But where the
build environment has installed binman separate, it knows better.

So I don't think this is a board config thing,

>
> [snip]
> > +The tools-only build bytes pylibfdt by default. To disable this, use the
>
> "build bytes" should probably be "builds" I suspect.

Yes, thanks.

Regards,
Simon