Hi Heinrich,
On Mon, 26 Sept 2022 at 00:35, Heinrich Schuchardt <xypron.g...@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 <s...@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.
Regards,
Simon
