On 31 October 2012 15:21, Chad Versace <span dir="ltr"><<a href="mailto:chad.versace@linux.intel.com" target="_blank">chad.versace@linux.intel.com</a>></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">
Tapani, see the bottom of the message.<br>
<div class="im"><br>
On 10/30/2012 10:42 AM, Paul Berry wrote:<br>
> Previously dispatch table-related code was generated from gl_API.xml,<br>
> so it did not include slots for GLES1-only functions (such as those<br>
> taking fixed-point arguments).<br>
><br>
> This patch generates dispatch table-related code from<br>
> gl_and_es_API.xml, so that GLES1-only functions are included.  This<br>
> paves the way for future patches that will unify the GLES1 dispatch<br>
> table with the dispatch tables for the other APIs.<br>
><br>
> The following generated files are affected:<br>
> - glapi_x86.S<br>
> - glapi_x86-64.S<br>
> - glapi_sparc.S<br>
> - glprocs.h<br>
> - glapitemp.h<br>
> - glapitable.h<br>
> - glapi_gentable.c<br>
> - dispatch.h<br>
> - remap_helper.h<br>
><br>
> Since this change affects makefiles, a full rebuild is required.<br>
> ---<br>
>  src/mapi/glapi/SConscript      |  6 +++---<br>
>  src/mapi/glapi/gen/Makefile.am | 18 +++++++++---------<br>
>  src/mapi/glapi/gen/SConscript  | 10 +++++-----<br>
>  3 files changed, 17 insertions(+), 17 deletions(-)<br>
><br>
<br>
</div><div class="im">> diff --git a/src/mapi/glapi/gen/Makefile.am b/src/mapi/glapi/gen/Makefile.am<br>
> index 40aaf51..24bdbaf 100644<br>
> --- a/src/mapi/glapi/gen/Makefile.am<br>
> +++ b/src/mapi/glapi/gen/Makefile.am<br>
> @@ -187,27 +187,27 @@ $(MESA_GLAPI_DIR)/glapi_mapi_tmp.h: $(MESA_MAPI_DIR)/mapi_abi.py $(COMMON_ES)<br>
>               --printer glapi --mode lib $(srcdir)/gl_and_es_API.xml > $@<br>
><br>
<br>
</div>I see a dependency bug. The rules below depend on list COMMON, which does not<br>
contain gl_and_es_API.xml. I think the best way to fix this is to remove the<br>
list COMMON_ES and fold its contents into COMMON.<br></blockquote><div><br>Oops, good catch.  I'll make that fix and send out a v2 of this patch.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<div class="im"><br>
>  $(MESA_GLAPI_DIR)/glprocs.h: gl_procs.py $(COMMON)<br>
> -     $(PYTHON_GEN) $< -f $(srcdir)/gl_API.xml > $@<br>
> +     $(PYTHON_GEN) $< -f $(srcdir)/gl_and_es_API.xml > $@<br>
><br>
>  $(MESA_GLAPI_DIR)/glapitemp.h: gl_apitemp.py $(COMMON)<br>
> -     $(PYTHON_GEN) $< -f $(srcdir)/gl_API.xml > $@<br>
> +     $(PYTHON_GEN) $< -f $(srcdir)/gl_and_es_API.xml > $@<br>
><br>
>  $(MESA_GLAPI_DIR)/glapitable.h: gl_table.py $(COMMON)<br>
> -     $(PYTHON_GEN) $< -f $(srcdir)/gl_API.xml > $@<br>
> +     $(PYTHON_GEN) $< -f $(srcdir)/gl_and_es_API.xml > $@<br>
><br>
>  $(MESA_GLAPI_DIR)/glapi_gentable.c: gl_gentable.py $(COMMON)<br>
> -     $(PYTHON_GEN) $< -f $(srcdir)/gl_API.xml > $@<br>
> +     $(PYTHON_GEN) $< -f $(srcdir)/gl_and_es_API.xml > $@<br>
><br>
<br>
<br>
</div>This patch doesn't touch the Android makefiles, and I think that's ok. Still,<br>
I want to verify with Tapani that this won't break the Android build before<br>
comitting.<br>
<br>
</blockquote></div><br>Yeah, I don't really understand how the Android build process handles these generated files--they don't seem to appear in any of the Android.mk files.  Tapani, I'm hoping I can commit these patches tomorrow morning (pacific time) so I can send out another patch series that depends on them.  Do you think you will have time to try the series out before then?<br>