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

Chad Versace chad.versace at linux.intel.com
Mon Aug 18 11:40:07 PDT 2014


On 08/18/2014 11:15 AM, Kristian Høgsberg wrote:
> On Sun, Aug 17, 2014 at 5:32 PM, Daniel Kurtz <djkurtz at chromium.org> wrote:
>>
>> On Aug 18, 2014 3:49 AM, "Kristian Høgsberg" <krh at bitplanet.net> wrote:

>>> It certainly shouldn't be in the header, but egl-util.c needs it and
>>> breaks if you remove it.
>>
>> egl-util.c builds fine for GLES without <GL/gl.h> (and breaks with it).  Why
>> is it needed for GL?  What am I missing?

I think this file used to require <GL/gl.h> before the EGL tests began using piglit-dispatch.
> 
> I got compile errors that looked like a missing Xlib.h include, not
> gl.h.  Trying again it was just Xlib deprecation warnings, and they're
> there without your patch.  Sorry for the false alarm.
> 
> Reviewed-by: Kristian Høgsberg <krh at bitplanet.net>

Looks good to me too.
Reviewed-by: Chad Versace <chad.versace at linux.intel.com>


More information about the Piglit mailing list