Re: [PATCH] cirrus.yml: Cover SPICE in the MSYS2 job

[Thomas Huth]

On 22/03/2023 13.13, Philippe Mathieu-Daudé wrote:
> On 22/3/23 12:30, Thomas Huth wrote:
> > On 22/03/2023 12.05, Philippe Mathieu-Daudé wrote:
> > > Include the mingw-w64-x86_64-spice package so SPICE is covered:
> > >
> > >    C compiler for the host machine: cc -m64 -mcx16 (gcc 12.2.0 "cc 
> > > (Rev10, Built by MSYS2 project) 12.2.0")
> > >    ...
> > >    Run-time dependency spice-protocol found: YES 0.14.4
> > >    Run-time dependency spice-server found: YES 0.15.1
> > >
> > > In particular this would have helped catching the build issue
> > > reported as https://gitlab.com/qemu-project/qemu/-/issues/1553:
> >
> > Well, hardly anybody is looking at the output on cirrus-ci.com, so this 
> > patch here likely would not have prevented the merging of this bug... for 
> > that, you rather have to add it to .gitlab-ci.d/windows.yml instead.
>
> It seems we have 2 jobs doing the same testing (even same version,
> msys2-base-x86_64-20220603.sfx.exe). Does it make sense to maintain
> both? At least with GitLab we notice a failure.

We can test more things (targets) in the Cirrus-CI job since the build 
machines are way more powerful there ... but yes, since people rarely look 
at this output, it's maybe better to disable them and focus on the gitlab-CI 
jobs instead.

 Thomas