[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 1/2] gitlab: exclude sparc-softmmu and riscv32-softmmu from c
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH 1/2] gitlab: exclude sparc-softmmu and riscv32-softmmu from cross builds |
Date: |
Wed, 18 Aug 2021 10:29:00 +0100 |
User-agent: |
Mutt/2.0.7 (2021-05-04) |
On Wed, Aug 18, 2021 at 10:15:47AM +0100, Mark Cave-Ayland wrote:
> On 10/08/2021 15:06, Daniel P. Berrangé wrote:
>
> > We need to cut down compile time by excluding more targets. Both these
> > targets still have their 64-bit variant enabled, so the loss of coverage
> > is mitigated to some degree.
> >
> > Signed-off-by: Daniel P. Berrangé <berrange@redhat.com>
> > ---
> > .gitlab-ci.d/crossbuild-template.yml | 3 ++-
> > 1 file changed, 2 insertions(+), 1 deletion(-)
> >
> > diff --git a/.gitlab-ci.d/crossbuild-template.yml
> > b/.gitlab-ci.d/crossbuild-template.yml
> > index 7d3ad00a1e..cfb576b54c 100644
> > --- a/.gitlab-ci.d/crossbuild-template.yml
> > +++ b/.gitlab-ci.d/crossbuild-template.yml
> > @@ -9,7 +9,8 @@
> > ../configure --enable-werror --disable-docs $QEMU_CONFIGURE_OPTS
> > --disable-user --target-list-exclude="arm-softmmu cris-softmmu
> > i386-softmmu microblaze-softmmu mips-softmmu mipsel-softmmu
> > - mips64-softmmu ppc-softmmu sh4-softmmu xtensa-softmmu"
> > + mips64-softmmu ppc-softmmu riscv32-softmmu sh4-softmmu
> > + sparc-softmmu xtensa-softmmu"
> > - make -j$(expr $(nproc) + 1) all check-build $MAKE_CHECK_ARGS
> > - if grep -q "EXESUF=.exe" config-host.mak;
> > then make installer;
>
> I'd prefer to keep sparc-softmmu if possible, simply because my everyday
> platform is Linux and so having a cross-build for Windows will catch things
> that I may miss on a day-to-day basis. Is sparc-softmmu currently enabled as
> part of the native MINGW64 build?
Note the builds still include 'sparc64-softmmu', so we're only loosing
cover in places where it diverges fromthe 64-bit build, but this will
sstil get coverage in native builds.
> If I go to my Gitlab QEMU fork Settings -> CI/CD -> Variables there is an
> option to set variables that can be used in job scripts. Perhaps this could
> be used so that I can configure my personal QEMU fork to always run
> sparc-softmmu builds when preparing PRs even if they aren't enabled for
> everyone by default? At least this would then allow me to spot any breakage
> before sending a final PR to Peter.
Separately from this I'm doing some work to make things more configurable
https://lists.gnu.org/archive/html/qemu-devel/2021-08/msg02102.html
but not on this level of granularity.
Rather than globally excluding from all cross builds, I think we ought to
split it up more equitably acrss the different builds in some way.
Regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
[PATCH 2/2] gitlab: skip many more targets in windows cross builds, Daniel P . Berrangé, 2021/08/10
Re: [PATCH 0/2] gitlab: avoid timeouts in windows cross builds, Thomas Huth, 2021/08/11