[Buildroot] [PATCH v1] qt5multimedia: fix plugins videonode egl x11 header related compile failure
Thomas Petazzoni
thomas.petazzoni at free-electrons.com
Mon Jul 3 19:02:40 UTC 2017
Hello,
On Mon, 3 Jul 2017 20:46:54 +0200, Peter Seiderer wrote:
> On Sun, 2 Jul 2017 23:14:49 +0200, Thomas Petazzoni <thomas.petazzoni at free-electrons.com> wrote:
>
> > Hello,
> >
> > On Sun, 2 Jul 2017 21:01:27 +0200, Peter Seiderer wrote:
> >
> > > Same fixing pattern as for qt5base (and for qtbase accepted upstream, see e.g. [1]),
> >
> > Well, [1] is about adding QT_EGL_NO_X11, which is a QT definition. It
> > is very different from MESA_EGL_NO_X11_HEADERS, which is an
> > OpenGL-level definition.
>
> The missing link is in file src/platformsupport/eglconvenience/qt_egl_p.h line 54ff:
>
> 54 #ifdef QT_EGL_NO_X11
> 55 # define MESA_EGL_NO_X11_HEADERS // MESA
> 56 # define WIN_INTERFACE_CUSTOM // NV
> 57 #endif // QT_EGL_NO_X11
> 58
> 59 #ifdef QT_EGL_WAYLAND
> 60 # define WAYLAND // NV
> 61 #endif // QT_EGL_WAYLAND
> 62
> 63 #include <EGL/egl.h>
> 64 #include <EGL/eglext.h>
>
> QT_EGL_NO_X11 forces MESA_EGL_NO_X11_HEADERS before the egl headers are included...
Ah, okay, indeed. But still, I really think it should be the
responsibility of the OpenGL provider to define MESA_EGL_NO_X11_HEADERS
in its .pc files, rather than having this logic duplicated in all
packages "consuming" the OpenGL API.
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux and Kernel engineering
http://free-electrons.com
More information about the buildroot
mailing list