Re: [PULL v2 0/6] testing updates

[Thomas Huth]

On 03/01/2023 18.47, Alex Bennée wrote:
> Peter Maydell <peter.maydell@linaro.org> writes:
>
> > On Fri, 23 Dec 2022 at 17:21, Alex Bennée <alex.bennee@linaro.org> wrote:
> > > The following changes since commit 222059a0fccf4af3be776fe35a5ea2d6a68f9a0b:
> > >
> > >    Merge tag 'pull-ppc-20221221' of https://gitlab.com/danielhb/qemu
> > > into staging (2022-12-21 18:08:09 +0000)
> > >
> > > are available in the Git repository at:
> > >
> > >    https://gitlab.com/stsquad/qemu.git tags/pull-testing-next-231222-1
> > >
> > > for you to fetch changes up to 3b4f911921e4233df0ba78d4acd2077da0b144ef:
> > >
> > >    gitlab-ci: Disable docs and GUIs for the build-tci and
> > > build-tcg-disabled jobs (2022-12-23 15:17:13 +0000)
> > >
> > > ----------------------------------------------------------------
> > > testing updates:
> > >
> > >    - fix minor shell-ism that can break check-tcg
> > >    - turn off verbose logging on custom runners
> > >    - make configure echo call in CI
> > >    - fix unused variable in linux-test
> > >    - add binary compiler docker image for hexagon
> > >    - disable doc and gui builds for tci and disable-tcg builds
> > >
> > > ----------------------------------------------------------------
> >
> > The msys2-64bit job failed with a weird 'missing terminating "'
> > compiler error on ui/shader/texture-blit-vert.h:
> >
> > https://gitlab.com/qemu-project/qemu/-/jobs/3547023328
> >
> > as did msys2-32-bit:
> >
> > https://gitlab.com/qemu-project/qemu/-/jobs/3547023329
> >
> > Any idea? Is this just an existing error that's been masked
> > by the msys2 jobs consistently timing out before they get
> > to it ?
>
> It ran fine when I pushed the branch:
>
>    https://gitlab.com/stsquad/qemu/-/jobs/3519255299
>
> but I can't see it building the object file ui_shader.c.obj in my build
> so I wonder if the builder has updated its development libraries since?
> I think it is a generated file.

FWIW, I also ran into this problem independently from this pull request 
while trying to increase the timeout of the jobs to 80 minutes:

 https://gitlab.com/thuth/qemu/-/jobs/3546685202
 https://gitlab.com/thuth/qemu/-/jobs/3546685203

So I don't think it is related to this pull request but something different.

Diff'ing the output between Alex' and my run, I can see that Alex log has:

 Has header "epoxy/egl.h" with dependency epoxy: NO
 ...
 OpenGL support (epoxy)       : NO

While my broken run has:

 Has header "epoxy/egl.h" with dependency epoxy: YES
 ...
 OpenGL support (epoxy)       : YES 1.5.10

So seems like something changed in the build environment, so that epoxy is 
now enabled there.

I guess we should disable it manually there again for the time being, until 
someone has figured out the real fix?

 Thomas