Re: [PATCH v2 1/2] tests/docker: gcc-10 based images for ppc64{, le} tests

[Matheus K. Ferst]

On 23/04/2021 09:54, Fabiano Rosas wrote:
> matheus.ferst@eldorado.org.br writes:
>
> > From: Matheus Ferst <matheus.ferst@eldorado.org.br>
> >
> > A newer compiler is needed to build tests for Power10 instructions. As
> > done for arm64 on c729a99d2701, a new '-test-cross' image is created for
> > ppc64 and ppc64le. As done on 936fda4d771f, a test for compiler support
> > is added to verify that the toolchain in use has '-mpower10'. Finally,
> > unused images (docker-power-cross and docker-ppc64-cross) are removed.
> >
> > Signed-off-by: Matheus Ferst <matheus.ferst@eldorado.org.br>
> > ---
> >   .gitlab-ci.d/containers.yml                   | 11 ++--------
> >   tests/docker/Makefile.include                 |  5 ++---
> >   .../dockerfiles/debian-powerpc-cross.docker   | 12 -----------
> >   .../debian-powerpc-test-cross.docker          | 17 ++++++++++++++++
> >   .../dockerfiles/debian-ppc64-cross.docker     | 11 ----------
> >   tests/tcg/configure.sh                        | 20 +++++++++----------
> >   6 files changed, 31 insertions(+), 45 deletions(-)
> >   delete mode 100644 tests/docker/dockerfiles/debian-powerpc-cross.docker
> >   create mode 100644 tests/docker/dockerfiles/debian-powerpc-test-cross.docker
> >   delete mode 100644 tests/docker/dockerfiles/debian-ppc64-cross.docker
> >
> > diff --git a/.gitlab-ci.d/containers.yml b/.gitlab-ci.d/containers.yml
> > index 33e4046e23..32f1b5887f 100644
> > --- a/.gitlab-ci.d/containers.yml
> > +++ b/.gitlab-ci.d/containers.yml
> > @@ -144,19 +144,12 @@ mipsel-debian-cross-container:
> >     variables:
> >       NAME: debian-mipsel-cross
> >   
> > -powerpc-debian-cross-container:
> > +powerpc-test-debian-cross-container:
> >     <<: *container_job_definition
> >     stage: containers-layer2
> >     needs: ['amd64-debian10-container']
>
> Should this be debian11?

Yes, I'll send a new version.

Thanks,
Matheus K. Ferst
Instituto de Pesquisas ELDORADO <http://www.eldorado.org.br/>
Analista de Software Júnior
Aviso Legal - Disclaimer <https://www.eldorado.org.br/disclaimer.html>