[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH v4 04/18] ui: avoid warnings about directdb on Alpine / musl
From: |
Daniel P . Berrangé |
Subject: |
Re: [PATCH v4 04/18] ui: avoid warnings about directdb on Alpine / musl libc |
Date: |
Wed, 24 Nov 2021 13:42:04 +0000 |
User-agent: |
Mutt/2.1.3 (2021-09-10) |
On Wed, Nov 24, 2021 at 02:32:56PM +0100, Thomas Huth wrote:
> On 24/11/2021 14.01, Daniel P. Berrangé wrote:
> > On Alpine, SDL is built with directfb support and this triggers warnings
> > during QEMU build
> >
> > In file included from /usr/include/directfb/direct/thread.h:38,
> > from /usr/include/directfb/direct/debug.h:43,
> > from /usr/include/directfb/direct/interface.h:36,
> > from /usr/include/directfb/directfb.h:49,
> > from /usr/include/SDL2/SDL_syswm.h:80,
> > from /builds/berrange/qemu/include/ui/sdl2.h:8,
> > from ../ui/sdl2-gl.c:31:
> > /usr/include/directfb/direct/os/waitqueue.h:41:25: error: redundant
> > redeclaration of 'direct_waitqueue_init' [-Werror=redundant-decls]
> > 41 | DirectResult DIRECT_API direct_waitqueue_init (
> > DirectWaitQueue *queue );
> > | ^~~~~~~~~~~~~~~~~~~~~
>
> I think that's a bug in the SDL2 package of Alpine. I've had trouble with
> that one in the past already ... so not sure whether it's our job now to
> work around bugs in that crappy Alpine package... I'd prefer if we simply do
> not compile-test SDL on Alpine instead.
It is a harmless bug in directfb headers, which has no ill
effects other than producing a warning when some app uses
-Wredundant-decls. Of course Alpine would ideally get this
patched, but at the same time it is reasonable for Alpine
users to want SDL and if someone has this package installed
already we don't want QEMU build to break due to -Werror.
Case in point, as we auto-generate dockerfiles to ensure
all pre-requisites are installed in all dockerfiles, the
Alpine dockerfile gains SDL and exposes this bug. We could
turn off -Werror but that's a overly big hammer compared
to this targetted fix.
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 v4 00/18] tests/docker: start using libvirt-ci's "lcitool" for dockerfiles, Daniel P . Berrangé, 2021/11/24
- [PATCH v4 01/18] ui: avoid compiler warnings from unused clipboard info variable, Daniel P . Berrangé, 2021/11/24
- [PATCH v4 02/18] spice: Update QXLInterface for spice >= 0.15.0, Daniel P . Berrangé, 2021/11/24
- [PATCH v4 03/18] meson: require liburing >= 0.3, Daniel P . Berrangé, 2021/11/24
- [PATCH v4 04/18] ui: avoid warnings about directdb on Alpine / musl libc, Daniel P . Berrangé, 2021/11/24
- [PATCH v4 05/18] ci: explicitly skip I/O tests on alpine, Daniel P . Berrangé, 2021/11/24
- [PATCH v4 06/18] tests/docker: switch fedora image to release 35, Daniel P . Berrangé, 2021/11/24
- [PATCH v4 07/18] tests: integrate lcitool for generating build env manifests, Daniel P . Berrangé, 2021/11/24
- [PATCH v4 08/18] tests/docker: auto-generate centos8.docker with lcitool, Daniel P . Berrangé, 2021/11/24
- [PATCH v4 10/18] tests/docker: auto-generate ubuntu1804.docker with lcitool, Daniel P . Berrangé, 2021/11/24
- [PATCH v4 09/18] tests/docker: auto-generate fedora.docker with lcitool, Daniel P . Berrangé, 2021/11/24
- [PATCH v4 11/18] tests/docker: auto-generate ubuntu2004.docker with lcitool, Daniel P . Berrangé, 2021/11/24