[PATCH 09/45] CI: Install pyelftools for builds

Tom Rini trini at konsulko.com
Fri Sep 30 14:56:24 CEST 2022


On Thu, Sep 29, 2022 at 05:55:54PM -0600, Simon Glass wrote:
> Hi Tom,
> 
> On Thu, 29 Sept 2022 at 09:05, Tom Rini <trini at konsulko.com> wrote:
> >
> > On Wed, Sep 28, 2022 at 04:20:41AM -0600, Simon Glass wrote:
> > > Hi Heinrich,
> > >
> > > On Mon, 26 Sept 2022 at 00:35, Heinrich Schuchardt <xypron.glpk at gmx.de> wrote:
> > > >
> > > > On 9/25/22 17:02, Simon Glass wrote:
> > > > > Binman needs this module to build sandbox_vpl so install it in the
> > > > > before_script and in the world build. The existing pip install is too
> > > > > late for the buildman invocation.
> > > > >
> > > > > Add it to the docker file for the future.
> > > > >
> > > > > Signed-off-by: Simon Glass <sjg at chromium.org>
> > > > > ---
> > > > >
> > > > >   .gitlab-ci.yml          | 2 ++
> > > > >   tools/docker/Dockerfile | 1 +
> > > > >   2 files changed, 3 insertions(+)
> > > > >
> > > > > diff --git a/.gitlab-ci.yml b/.gitlab-ci.yml
> > > > > index 8e94bf8d4e8..47e36edcc4a 100644
> > > > > --- a/.gitlab-ci.yml
> > > > > +++ b/.gitlab-ci.yml
> > > > > @@ -34,6 +34,7 @@ stages:
> > > > >       # If we've been asked to use clang only do one configuration.
> > > > >       - export UBOOT_TRAVIS_BUILD_DIR=/tmp/${TEST_PY_BD}
> > > > >       - echo BUILD_ENV ${BUILD_ENV}
> > > > > +    - pip install pyelftools
> > > >
> > > > Why do we need pip install here? Is pyelftools used outside the Docker
> > > > image to which you add pyelftools below?
> > >
> > > We don't need it if Tom wants to regen the docker image, but I'm not
> > > sure how often he does that. So this change actually makes CI work, in
> > > either case.
> >
> > I would re-generate the CI image as part of the series, and we should
> > really do this separate and drop the pip install pyelftools we already
> > have, too.
> 
> OK, shall I resend this patch accordingly or will you just change it?
> Or should I split into two patches, one to add to Dockerfile (I hate
> the capital D!) and one to remove the pip install?

In v2 of the series, doing all of this as patch #1 will make my life
easier, how about that?

-- 
Tom
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 659 bytes
Desc: not available
URL: <https://lists.denx.de/pipermail/u-boot/attachments/20220930/8dc0b273/attachment.sig>


More information about the U-Boot mailing list