<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2016-09-07 12:56 GMT+02:00 Emil Velikov <span dir="ltr"><<a href="mailto:emil.l.velikov@gmail.com" target="_blank">emil.l.velikov@gmail.com</a>></span>:<br><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div class="gmail-HOEnZb"><div class="gmail-h5">On 7 September 2016 at 11:01, Mauro Rossi <<a href="mailto:issor.oruam@gmail.com">issor.oruam@gmail.com</a>> wrote:<br>
> Hi Tapani,<br>
><br>
> does moving the declaration of the dependency to<br>
> src/mesa/drivers/dri/i965/<wbr>Android.mk also work?<br>
> Mauro<br>
><br>
> LOCAL_WHOLE_STATIC_LIBRARIES := \<br>
> $(MESA_DRI_WHOLE_STATIC_<wbr>LIBRARIES) \<br>
> + libmesa_genxml \<br>
> libmesa_isl<br>
><br>
><br>
</div></div>Mauro, it's very unlikely that this will work. Afaict the static lib<br>
is required to pull the generated XML headers into the generated C<br>
file.<br>
<br>
Adding libmesa_genxml to LOCAL_WHOLE_STATIC_LIBRARIES will move the<br>
dependency (and thus generation) of the said headers to the (final)<br>
link stage. At which point all the files are (should be) compiled.<br>
<br>
Tapani, please send a patch with your fix with a line worth of commit<br>
message (feel free to copy mine you want) ? Feel free to add<br>
Reviewed-by: Emil Velikov <<a href="mailto:emil.velikov@collabora.com">emil.velikov@collabora.com</a>><br></blockquote><div><br></div><div>Just a final check, minor aesthetics,</div><div><br></div><div>should the libmesa_genxml static library dependency be declared in <span style="color:rgb(80,0,80)">src/mesa/drivers/dri/i965/</span><wbr style="color:rgb(80,0,80)"><span style="color:rgb(80,0,80)">Android.mk</span></div><div><span style="color:rgb(80,0,80)">instead of doing it </span><span style="color:rgb(80,0,80)">src/mesa/drivers/dri/i965/</span><wbr style="color:rgb(80,0,80)"><span style="color:rgb(80,0,80)"><a href="http://Android.gen.mk">Android.gen.mk</a>, which sole purpose is to generate sources?</span></div><div><span style="color:rgb(80,0,80)"><br></span></div><div><span style="color:rgb(80,0,80)">M. </span></div></div><br></div></div>