On 24/05/2022 15.17, Konstantin Kostiuk wrote:
On Tue, May 24, 2022 at 4:13 PM Thomas Huth <th...@redhat.com
<mailto:th...@redhat.com>> wrote:
On 24/05/2022 15.00, Konstantin Kostiuk wrote:
>
>
>
>
> On Tue, May 24, 2022 at 1:24 PM Thomas Huth <th...@redhat.com
<mailto:th...@redhat.com>
> <mailto:th...@redhat.com <mailto:th...@redhat.com>>> wrote:
>
> On 24/05/2022 12.14, Marc-André Lureau wrote:
> > Hi
> >
> > On Tue, May 24, 2022 at 12:02 PM Konstantin Kostiuk
> <kkost...@redhat.com <mailto:kkost...@redhat.com>
<mailto:kkost...@redhat.com <mailto:kkost...@redhat.com>>> wrote:
> >>
> >> Hi Richard and Marc-André
> >>
> >> I looked into the compilation problem and have 2 solutions:
> >> 1. We can add some conditions to the win2qemu definition and
> >> skip NVME support when old mingw-headers are used.
> >> 2. We can bump the version of the Fedora docker image to 36 or 37
> >> that is used for cross-compilation tests.
> >>
> >> I think the second option is more valuable because we remove
> >> pregenerated qga-vss.tlb file and now we can check VSS build only
> >> at Fedora 37.
> >>
> >> What do you think?
> >
> > I'd try to do both: fix compilation with older headers, and
bump our
> > CI to f36. I don't know if our windows build environment has
strict
> > requirements like the unix/distro (build on old-stable for 2y).
>
> See
https://www.qemu.org/docs/master/about/build-platforms.html#windows
<https://www.qemu.org/docs/master/about/build-platforms.html#windows>
>
<https://www.qemu.org/docs/master/about/build-platforms.html#windows
<https://www.qemu.org/docs/master/about/build-platforms.html#windows>> :
>
> "The project supports building QEMU with current versions of the
MinGW
> toolchain, either hosted on Linux (Debian/Fedora) or via MSYS2 on
Windows."
>
> Since Fedora 35 is still a supported build host, I think you
should make
> sure that it works with the MinGW toolchain from that distro, too.
>
>
> Currently, CI uses Fedora 33 which is already EOL. Fedora 35 has updated
> mingw-headers and the current version of code compiles without any
errors.
> So if we want to support only Fedora 35+, we can just bump the CI
docker image.
Ah, right, I was looking at the wrong file. So yes, in that case, please
simply update the docker image.
What about Debian (since this is mentioned on the support page, too)? I
think we don't have to worry about Debian 10 anymore, since Debian 10 will
already be EOL once we release QEMU 7.1 ... but what about Debian 11? Do
the
MinGW packages there contain the updated headers, too?
As I know we do not test cross-compilation at Debian. Debian does not have
even mingw-glib2. Debian only has the mingw-gcc toolkit.
Oh, interesting! Then I wonder why Debian is mentioned there ... seems like
it has been added here:
https://git.qemu.org/?p=qemu.git;a=commitdiff;h=e6e80fcfd6c47823
Daniel, do you remember whether we supported Debian for MinGW
cross-compilation in the past?
Thomas
