On 16 March 2012 11:52, Chad Versace <span dir="ltr">&lt;<a href="mailto:chad.versace@linux.intel.com">chad.versace@linux.intel.com</a>&gt;</span> wrote:<br><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="im">On 03/12/2012 02:41 PM, Paul Berry wrote:<br>
&gt; Previous patches set up the piglit-dispatch infrastructure but did not<br>
&gt; use it.  This patch enables piglit-dispatch and uses it instead of<br>
&gt; GLEW.<br>
&gt;<br>
&gt; No piglit regressions on Intel SandyBridge (Linux) or Mac OSX.<br>
<br>
</div>Nice. Only one more platform to conquer.<br>
<div class="im"><br>
&gt;  #if defined(USE_OPENGL)<br>
&gt; -#    include &quot;glew.h&quot;<br>
&gt; -     /* Include the real headers too, in case GLEW misses something. */<br>
&gt; +#    include &quot;piglit-dispatch.h&quot;<br>
&gt; +     /* Include the real headers too, in case piglit-dispatch misses something. */<br>
&gt;  #    ifdef __APPLE__<br>
&gt;  #            include &lt;OpenGL/gl.h&gt;<br>
&gt;  #            include &lt;OpenGL/glu.h&gt;<br>
<br>
</div>Shouldn&#39;t Apple&#39;s &lt;OpenGL/gl.h&gt; be removed too?<br>
I think we discussed this before, but I don&#39;t remember the conclusion.<br></blockquote><div><br>Yes, you&#39;re right.  In fact this whole block should be removed:<br><br>    /* Include the real headers too, in case piglit-dispatch misses something. */<br>
#    ifdef __APPLE__<br>#        include &lt;OpenGL/gl.h&gt;<br>#        include &lt;OpenGL/glu.h&gt;<br>#        include &lt;OpenGL/glext.h&gt;<br>#    else<br>#        include &lt;GL/gl.h&gt;<br>#        include &lt;GL/glu.h&gt;<br>
#        include &lt;GL/glext.h&gt;<br>#    endif<br><br>I&#39;ll update the patch.<br><br></div></div>