<div dir="ltr">Hi Emil,<div><br></div><div>I checked with Chih-Wei and <span style="font-size:12.8px">LOCAL_EXPORT_C_INCLUDE_DIRS is the preferred way.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">I took the chance to finally understand why the single line of change you proposed will suffice,</span></div><div><span style="font-size:12.8px">it's because glsl_compiler module invokes libmesa_glsl as static library and the same applies to i965_dri module.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">My last problem was probably due to uncleaned mesa targets folders in $OUT path,</span></div><div><span style="font-size:12.8px">now I have checked again and building errors are gone.</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Thanks a lot Emil and Chih-Wei</span></div><div><span style="font-size:12.8px"><br></span></div><div><span style="font-size:12.8px">Mauro</span></div></div><div class="gmail_extra"><br><div class="gmail_quote">2015-11-08 14:12 GMT+01: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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi Mauro,<br>
<span class=""><br>
On 8 November 2015 at 12:59, Mauro Rossi <<a href="mailto:issor.oruam@gmail.com">issor.oruam@gmail.com</a>> wrote:<br>
> Update2: I'm getting the building error in both x86 target and x86_64<br>
> target.<br>
><br>
> I'm relieved it's not arch dependend, I suspect that export will require a<br>
> dependency to be declared,<br>
>  because if i965_dri module is built before glsl ones we will have the<br>
> error.<br>
><br>
> The LOCAL_C_INCLUDES even if not elegant, avoided the problem in the first<br>
> place,<br>
> but I'd like to learn the by the best practice and apply it in the future.<br>
><br>
</span>Fwiw I'm all for LOCAL_C_INCLUDES (I even mentioned a few times a way<br>
that we can share those and minimise these issues), but I believe<br>
Chih-Wei was not really a fan of them. If he's ok with it I'll push<br>
your original patch.<br>
<br>
Regards,<br>
Emil<br>
<br>
P.S. Typos - for each one we fix, another we introduce another one or more  :-P<br>
</blockquote></div><br></div>