[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