[PATCH v3 4/6] buildman: Always use the full path in CROSS_COMPILE

Simon Glass sjg at chromium.org
Mon Jul 15 15:31:21 CEST 2024


On Wed, Jun 26, 2024 at 09:00:43AM +0100, Simon Glass wrote:
> Hi Andrejs,
>
> On Tue, 25 Jun 2024 at 16:16, Andrejs Cainikovs
> <andrejs.cainikovs at toradex.com> wrote:
> >
> > On Tue, Jun 25, 2024 at 01:38:07PM +0100, Simon Glass wrote:
> > > Hi Andrejs,
> > >
> > > On Tue, 25 Jun 2024 at 01:06, Andrejs Cainikovs
> > > <andrejs.cainikovs at toradex.com> wrote:
> > > >
> > > > On Sun, Jun 23, 2024 at 11:56:20AM -0600, Simon Glass wrote:
> > > > > The feature to set the toolchain path does not seem to be needed. It
> > > > > causes problems with venv (see [1]). Let's remove it.
> > > > >
> > > > > Add some tests while we are here.
> > > > >
> > > > > It does not look like any docs changes are needed for this.
> > > > >
> > > > > [1] https://patchwork.ozlabs.org/project/uboot/patch/20240621131423.2363294-6-sjg@chromium.org/
> > > > >
> > > > > Signed-off-by: Simon Glass <sjg at chromium.org>
> > > > > Suggested-by: Tom Rini <trini at konsulko.com>
> > > > > ---
> > > > >
> > > > > Changes in v3:
> > > > > - Drop the PATH modification altogether
> > > > >
> > > > >  tools/buildman/bsettings.py     |  3 ++
> > > > >  tools/buildman/builder.py       |  5 +--
> > > > >  tools/buildman/builderthread.py |  4 +-
> > > > >  tools/buildman/cmdline.py       |  2 -
> > > > >  tools/buildman/control.py       |  6 +--
> > > > >  tools/buildman/test.py          | 75 +++++++++++++++++++++++++++++++++
> > > > >  tools/buildman/toolchain.py     | 20 ++++-----
> > > > >  7 files changed, 92 insertions(+), 23 deletions(-)
> > > > >
Applied to u-boot-dm, thanks!


More information about the U-Boot mailing list