<div dir="auto"><div><br><div class="gmail_extra"><br><div class="gmail_quote">Il 18/dic/2017 17:27, "Emil Velikov" <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>> ha scritto:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="elided-text">On 17 December 2017 at 23:34, Mauro Rossi <<a href="mailto:issor.oruam@gmail.com">issor.oruam@gmail.com</a>> wrote:<br>
> target.c requires "xmlpool/options.h" generated header<br>
> or the following tricky Android building error may appear:<br>
><br>
> In file included from external/mesa/src/gallium/<wbr>targets/dri/target.c:1:<br>
> In file included from external/mesa/src/gallium/<wbr>auxiliary/target-helpers/drm_<wbr>helper.h:185:<br>
> out/target/product/x86_64/gen/<wbr>STATIC_LIBRARIES/libmesa_pipe_<wbr>radeonsi_intermediates/<wbr>radeonsi/si_driinfo.h:19:7: error: expected '}'<br>
>       DRI_CONF_RADEONSI_ENABLE_<wbr>SISCHED("false")<br>
>       ^<br>
> external/mesa/src/gallium/<wbr>auxiliary/target-helpers/drm_<wbr>helper.h:182:55: note: to match this '{'<br>
>    static const struct drm_conf_ret xml_options_ret = {<br>
>                                                       ^<br>
> 1 error generated.<br>
><br>
> Fixes: 0f8c5de869 ("radeonsi: prepare for driver-specific driconf options")<br>
><br>
> Cc: "17.3" <<a href="mailto:mesa-stable@lists.freedesktop.org">mesa-stable@lists.<wbr>freedesktop.org</a>><br>
> ---<br>
>  src/gallium/targets/dri/<wbr>Android.mk | 4 ++--<br>
>  1 file changed, 2 insertions(+), 2 deletions(-)<br>
><br>
> diff --git a/src/gallium/targets/dri/<wbr>Android.mk b/src/gallium/targets/dri/<wbr>Android.mk<br>
> index d9923043af..a5b5f92ea0 100644<br>
> --- a/src/gallium/targets/dri/<wbr>Android.mk<br>
> +++ b/src/gallium/targets/dri/<wbr>Android.mk<br>
> @@ -26,10 +26,10 @@ LOCAL_PATH := $(call my-dir)<br>
>  include $(CLEAR_VARS)<br>
><br>
>  LOCAL_MODULE := gallium_dri<br>
> -<br>
> +LOCAL_MODULE_CLASS := SHARED_LIBRARIES<br>
>  LOCAL_MODULE_RELATIVE_PATH := $(MESA_DRI_MODULE_REL_PATH)<br>
>  LOCAL_SRC_FILES := target.c<br>
> -<br>
> +LOCAL_C_INCLUDES := $(call generated-sources-dir-for,<wbr>STATIC_LIBRARIES,libmesa_util,<wbr>,)<br>
>  LOCAL_CFLAGS :=<br>
><br>
</div>This is slightly confusing, quick grep shows:<br>
./src/util/Android.mk<br>
...<br>
LOCAL_MODULE := libmesa_util<br>
...<br>
LOCAL_EXPORT_C_INCLUDE_DIRS := $(intermediates)<br>
...<br>
<br>
src/gallium/targets/dri/<wbr>Android.mk<br>
LOCAL_WHOLE_STATIC_LIBRARIES := \<br>
    ...<br>
    libmesa_util \<br>
    ...<br>
<br>
src/gallium/targets/dri/<wbr>Android.mk-     libmesa_loader<br>
<br>
- libmesa_util already exports</blockquote><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br></blockquote><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">AKA</blockquote><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
 - static lib exports the includes<br>
 - foo_dri.so already pulls the static lib<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">All confirmed and it's strange to us too, but when checking out Mesa 17.3 and 17.4 branches we, i.e. me and Quiang Yu, get the error.</div><div dir="auto"><br></div><div dir="auto">Maybe Qiang could explain, I just added the only possibile dependency.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
+ misc: if LOCAL_MODULE_CLASS is omitted we default to SHARED_LIBRARIES<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">LOCAL_MODULE_CLASS is required to be able to use <span style="font-family:sans-serif">generated-sources-dir-for macro</span></div><div dir="auto"><span style="font-family:sans-serif"><br></span></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Can you confirm the above are present in the tree you're using? Did it<br>
work with older Android - aka, something in their system broke?<br></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">Same issue happening with Nougat, when checking out 17.3 or 17.4 </div><div dir="auto">Cheers</div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Thanks<br>
<font color="#888888">Emil<br>
</font></blockquote></div><br></div></div></div>