Re: [PATCH 2/2] gitlab: Add cross-riscv64-system, cross-riscv64-user

[Richard Henderson]

On 9/15/21 8:03 AM, Philippe Mathieu-Daudé wrote:
> On 9/14/21 8:58 PM, Richard Henderson wrote:
> > Signed-off-by: Richard Henderson <richard.henderson@linaro.org>
> > ---
> >   .gitlab-ci.d/crossbuilds.yml | 14 ++++++++++++++
> >   1 file changed, 14 insertions(+)
> >
> > diff --git a/.gitlab-ci.d/crossbuilds.yml b/.gitlab-ci.d/crossbuilds.yml
> > index f10168db2e..0fe4a55ac5 100644
> > --- a/.gitlab-ci.d/crossbuilds.yml
> > +++ b/.gitlab-ci.d/crossbuilds.yml
> > @@ -124,6 +124,20 @@ cross-ppc64el-user:
> >     variables:
> >       IMAGE: debian-ppc64el-cross
> >   
> > +cross-riscv64-system:
> > +  extends: .cross_system_build_job
> > +  needs:
> > +    job: riscv64-debian-cross-container
> > +  variables:
> > +    IMAGE: debian-riscv64-cross
> > +
> > +cross-riscv64-user:
> > +  extends: .cross_user_build_job
> > +  needs:
> > +    job: riscv64-debian-cross-container
> > +  variables:
> > +    IMAGE: debian-riscv64-cross
> > +
>
> Pending discussion on patch #1 of this series, I believe
> this job is likely going to fail, so must use the
> 'allow_failure: true' tag or something else (manual?).
> Meanwhile this is sorted out:
> NAcked-by: Philippe Mathieu-Daudé <f4bug@amsat.org>

My only question is whether it's the image creation step that should be allowed to fail. 
If the dpkg deps weren't so broken as to not install, I *think* we should be able to rely 
on the result for the cross-build.

At least that's how I'd prefer to arrange things, if it works...

r~