<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div class="h5">
> +eglinterop_HEADERS = $(top_srcdir)/include/GL/mesa_<wbr>glinterop.h<br>
</div></div>IIRC Marek was pretty clear that this header should not be installed.<br>
Then again looking at our current wildcard installing ... seems like<br>
it was.<br>
<br>
Please drop this file from the install stage ?<br></blockquote><div><br></div><div>Dropped.<br> <br><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class="">> +if HAVE_COMMON_OSMESA<br>
> +osmesadir = $(includedir)/GL<br>
> +osmesa_HEADERS = $(top_srcdir)/include/GL/<wbr>osmesa.h<br>
> +endif<br>
> +<br>
</span>Why do we have this hunk, considering each target is handled explicitly ?<br>
<br>
IMHO we should drop either this or the similar ones in<br>
src/{mesa,gallium}/Makefile.<wbr>am. The latter might be better ?<br></blockquote><br>The idea was to seperate the interface install from the implementation  build in the same way that it's done for GLX.  Mostly to reduce code duplication.  At the top level the interface headers are installed then at the deeper nested level the chosen implementation is built.  It does, however, look like I forgot to drop the header install from the implementations.  Will do.<br></div><div class="gmail_quote"><div> <br></div></div></div></div>