[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] qemu-edid: drop cast
|
From: |
Daniel P . Berrangé |
|
Subject: |
Re: [PATCH] qemu-edid: drop cast |
|
Date: |
Tue, 13 Oct 2020 11:40:52 +0100 |
|
User-agent: |
Mutt/1.14.6 (2020-07-11) |
On Tue, Oct 13, 2020 at 12:34:46PM +0200, Gerd Hoffmann wrote:
> On Tue, Oct 13, 2020 at 10:23:22AM +0100, Daniel P. Berrangé wrote:
> > On Tue, Oct 13, 2020 at 11:16:15AM +0200, Gerd Hoffmann wrote:
> > > Not needed and makes some compilers error out with:
> > >
> > > qemu-edid.c:15:1: error: initializer element is not constant
> >
> > Which compilers do this. If we're not seeing this failure in our
> > CI, then we have a CI gap that likely needs filling.
>
> x86_64-w64-mingw32-gcc (GCC) 4.9.3 20150626 (Fedora MinGW 4.9.3-1.el7)
Oh, that's a really ancient mingw toolchain. Officially we only care
about current generation mingw toolchains ad don't try to support old
ones. Effectively this means whatever mingw is in current shipping
Fedora or MXE releases.
The fix is fine, but we don't need to worry about adding this CI after
all since its an unsupported platform.
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 :|