<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">2017-05-28 16:46 GMT+02:00 Rob Herring <span dir="ltr"><<a href="mailto:robh@kernel.org" target="_blank">robh@kernel.org</a>></span>:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On Fri, May 26, 2017 at 10:15 AM, Mauro Rossi <<a href="mailto:issor.oruam@gmail.com">issor.oruam@gmail.com</a>> wrote:<br>
> Fixes the following building error:<br>
><br>
> external/mesa/src/gallium/<wbr>drivers/svga/svga_screen.c:26:<wbr>10:<br>
> fatal error: 'git_sha1.h' file not found<br>
>          ^<br>
> 1 error generated.<br>
> ---<br>
>  src/gallium/drivers/svga/<wbr>Android.mk | 4 +++-<br>
>  1 file changed, 3 insertions(+), 1 deletion(-)<br>
><br>
> diff --git a/src/gallium/drivers/svga/<wbr>Android.mk b/src/gallium/drivers/svga/<wbr>Android.mk<br>
> index c50743d509..d19bd59bfe 100644<br>
> --- a/src/gallium/drivers/svga/<wbr>Android.mk<br>
> +++ b/src/gallium/drivers/svga/<wbr>Android.mk<br>
> @@ -30,7 +30,9 @@ include $(CLEAR_VARS)<br>
><br>
>  LOCAL_SRC_FILES := $(C_SOURCES)<br>
><br>
> -LOCAL_C_INCLUDES := $(LOCAL_PATH)/include<br>
> +LOCAL_C_INCLUDES := \<br>
> +       $(LOCAL_PATH)/include \<br>
> +       $(call generated-sources-dir-for,<wbr>STATIC_LIBRARIES,libmesa_<wbr>dricore,,)/main<br>
<br>
</span>Add this path to LOCAL_EXPORT_C_INCLUDE_DIRS in libmesa_dricore and<br>
add libmesa_dricore as a static lib here instead.<br>
<span class="HOEnZb"><font color="#888888"><br>
Rob<br></font></span></blockquote><div><br></div><div>Hi Rob, </div><div>just a few questions,</div><div><br></div><div>why should we export the header which is artificially associated to libmesa_dricore<br></div><div>and why treat this as a 'fake dependency'? The LOCAL_C_INCLUDE should suffice</div><div><br></div><div>What is benefit in doing this? What is the drawback in using the android macro created for this purpose?</div><div>Thanks</div><div><br></div><div>Mauro</div></div><br></div></div>