[Piglit] [PATCH] egl-util: remove GL include

[Ken Phillis Jr]

This patch looks good to me.
On Aug 17, 2014 7:39 AM, "Daniel Kurtz" <djkurtz at chromium.org> wrote:

> EGL util should not in any way depend on the headers of a particular
> client library.
>
> In fact, this breaks compilation on pure OpenGL ES systems, since they
> do not have OpenGL headers installed (<GL/gl.h>).
>
> Signed-off-by: Daniel Kurtz <djkurtz at chromium.org>
> ---
> I've test built this change on two different systems with EGL/OpenGL
> ES-only
> (ARM Mali and Nvidia Tegra).
>
> I have not built on a system with EGL + OpenGL, or mixed GL/GLES... does
> anybody have a system like that and can double check?
>
> Also, I do not have commit access, so can someone please submit this for
> me if
> it looks good?
>
> Thanks,
> -djk
>
>  tests/egl/egl-util.h | 1 -
>  1 file changed, 1 deletion(-)
>
> diff --git a/tests/egl/egl-util.h b/tests/egl/egl-util.h
> index f5c74fd..27d2926 100644
> --- a/tests/egl/egl-util.h
> +++ b/tests/egl/egl-util.h
> @@ -12,7 +12,6 @@
>  #include <X11/Xlib.h>
>  #include <X11/Xutil.h>
>  #include <X11/keysym.h>
> -#include <GL/gl.h>
>  #include <EGL/egl.h>
>  #include <EGL/eglext.h>
>
> --
> 2.1.0.rc2.206.gedb03e5
>
> _______________________________________________
> Piglit mailing list
> Piglit at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/piglit
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/piglit/attachments/20140817/6ebb1cab/attachment.html>