On 13/11/15 07:34 AM, ahmet acar wrote: > Hi. This fix that: > > https://bugs.freedesktop.org/show_bug.cgi?id=92104 >
This is a harmless solution to a disgusting problem with eglplatform.h Basically, if you have no X11 include files and you include eglplatform.h without -DMESA_EGL_NO_X11_HEADERS then eglplatform.h will try to #include X11/Xlib.h and X11/Xutil.h if you include wayland-egl.h first it will define WL_EGL_PLATFORM which shorts out a bunch of logic before it even gets to the EGL_NO_X11_HEADERS check. This patch is probably the least worst solution to the problem, so... pushed. Ahmet, I'll write a commit log and add the appropriate attribution for this patch, but in the future it'd make it easier to land your contributions if you set up git send-email. Thanks for sorting this out. Reviewed-by: Derek Foreman <der...@osg.samsung.com> (and pushed...) > --- > shared/platform.h | 1 + > 1 file changed, 1 insertion(+) > > diff --git a/shared/platform.h b/shared/platform.h > index 64e1fbf..cf4ecc0 100644 > --- a/shared/platform.h > +++ b/shared/platform.h > @@ -29,6 +29,7 @@ > #include <string.h> > > #ifdef ENABLE_EGL > +#include <wayland-egl.h> > #include <EGL/egl.h> > #include <EGL/eglext.h> > #endif > _______________________________________________ wayland-devel mailing list wayland-devel@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/wayland-devel