<font face="courier new,monospace"><br></font><br><div class="gmail_quote">On Thu, Oct 4, 2012 at 1:58 PM, Oliver McFadden <span dir="ltr"><<a href="mailto:oliver.mcfadden@linux.intel.com" target="_blank">oliver.mcfadden@linux.intel.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5">On Tue, Oct 02, 2012 at 04:20:51PM +0300, Negreanu, Adrian M wrote:<br>
> On Tue, Oct 2, 2012 at 3:01 PM, Tapani Pälli <<a href="mailto:tapani.palli@intel.com">tapani.palli@intel.com</a>> wrote:<br>
><br>
> > commit a010215463c63680c69e90202fe3fcd2e5b25fa6 removed ES2 specific<br>
> > dispatch<br>
> > table and remap_helper, since now we are using dispatch.h which is<br>
> > generated<br>
> > from gl_and_es_API.xml we need to generate a matching remap_helper using<br>
> > the<br>
> > same xml.<br>
> ><br>
> > Signed-off-by: Tapani Pälli <<a href="mailto:tapani.palli@intel.com">tapani.palli@intel.com</a>><br>
> > ---<br>
> >  src/mesa/<a href="http://Android.gen.mk" target="_blank">Android.gen.mk</a> | 2 +-<br>
> >  1 file changed, 1 insertion(+), 1 deletion(-)<br>
> ><br>
> > diff --git a/src/mesa/<a href="http://Android.gen.mk" target="_blank">Android.gen.mk</a> b/src/mesa/<a href="http://Android.gen.mk" target="_blank">Android.gen.mk</a><br>
> > index c17df1a..e5ce6ce 100644<br>
> > --- a/src/mesa/<a href="http://Android.gen.mk" target="_blank">Android.gen.mk</a><br>
> > +++ b/src/mesa/<a href="http://Android.gen.mk" target="_blank">Android.gen.mk</a><br>
> > @@ -136,7 +136,7 @@ $(intermediates)/main/dispatch.h: $(es_hdr_deps)<br>
> >         $(call es-gen, $* -m remap_table)<br>
> ><br>
> >  $(intermediates)/main/remap_helper.h: PRIVATE_SCRIPT := $(MESA_PYTHON2)<br>
> > $(glapi)/remap_helper.py<br>
> > -$(intermediates)/main/remap_helper.h: PRIVATE_XML := -f<br>
> > $(glapi)/gl_API.xml<br>
> > +$(intermediates)/main/remap_helper.h: PRIVATE_XML := -f<br>
> > $(glapi)/gl_and_es_API.xml<br>
> ><br>
> >  $(intermediates)/main/remap_helper.h: $(es_hdr_deps)<br>
> >         $(call es-gen, $*)<br>
> > --<br>
> ><br>
> 1.7.11.7<br>
> ><br>
> > _______________________________________________<br>
> > mesa-dev mailing list<br>
> > <a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
> > <a href="http://lists.freedesktop.org/mailman/listinfo/mesa-dev" target="_blank">http://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
> ><br>
><br>
><br>
> I tested it and it works fine;<br>
> the only thing is if doing an incremental build then remove<br>
> $ANDROID_OUT/libmesa_dricore_intermediates/main/remap_helper.h<br>
<br>
</div></div>It should not be required to manually remove the remap_helper.h file for<br>
an incremental build.  Breaking incremental builds is enough for me to<br>
NAK this patch.<br>
<br>
If the problem is after editing  gl_and_es_API.xml, remap_helper.h is<br>
not regenerated, then you have a missing dependency and need to fix the<br>
Makefile.</blockquote><div><br></div><div>mm -B did the trick</div><div><br></div><div>thanks </div></div><div><br></div>-- <br><div>Adrian Marius Negreanu</div><div>Open Source Technology Center <div><div><br></div><p></p>
</div></div><br>