Il 18/dic/2017 17:27, "Emil Velikov" <emil.l.veli...@gmail.com> ha scritto:
On 17 December 2017 at 23:34, Mauro Rossi <issor.or...@gmail.com> wrote:
> target.c requires "xmlpool/options.h" generated header
> or the following tricky Android building error may appear:
>
> In file included from external/mesa/src/gallium/targets/dri/target.c:1:
> In file included from external/mesa/src/gallium/
auxiliary/target-helpers/drm_helper.h:185:
> out/target/product/x86_64/gen/STATIC_LIBRARIES/libmesa_pipe_
radeonsi_intermediates/radeonsi/si_driinfo.h:19:7: error: expected '}'
> DRI_CONF_RADEONSI_ENABLE_SISCHED("false")
> ^
> external/mesa/src/gallium/auxiliary/target-helpers/drm_helper.h:182:55:
note: to match this '{'
> static const struct drm_conf_ret xml_options_ret = {
> ^
> 1 error generated.
>
> Fixes: 0f8c5de869 ("radeonsi: prepare for driver-specific driconf
options")
>
> Cc: "17.3" <mesa-sta...@lists.freedesktop.org>
> ---
> src/gallium/targets/dri/Android.mk | 4 ++--
> 1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/src/gallium/targets/dri/Android.mk b/src/gallium/targets/dri/
Android.mk
> index d9923043af..a5b5f92ea0 100644
> --- a/src/gallium/targets/dri/Android.mk
> +++ b/src/gallium/targets/dri/Android.mk
> @@ -26,10 +26,10 @@ LOCAL_PATH := $(call my-dir)
> include $(CLEAR_VARS)
>
> LOCAL_MODULE := gallium_dri
> -
> +LOCAL_MODULE_CLASS := SHARED_LIBRARIES
> LOCAL_MODULE_RELATIVE_PATH := $(MESA_DRI_MODULE_REL_PATH)
> LOCAL_SRC_FILES := target.c
> -
> +LOCAL_C_INCLUDES := $(call generated-sources-dir-for,
STATIC_LIBRARIES,libmesa_util,,)
> LOCAL_CFLAGS :=
>
This is slightly confusing, quick grep shows:
./src/util/Android.mk
...
LOCAL_MODULE := libmesa_util
...
LOCAL_EXPORT_C_INCLUDE_DIRS := $(intermediates)
...
src/gallium/targets/dri/Android.mk
LOCAL_WHOLE_STATIC_LIBRARIES := \
...
libmesa_util \
...
src/gallium/targets/dri/Android.mk- libmesa_loader
- libmesa_util already exports
AKA
- static lib exports the includes
- foo_dri.so already pulls the static lib
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.
Maybe Qiang could explain, I just added the only possibile dependency.
+ misc: if LOCAL_MODULE_CLASS is omitted we default to SHARED_LIBRARIES
LOCAL_MODULE_CLASS is required to be able to use generated-sources-dir-for
macro
Can you confirm the above are present in the tree you're using? Did it
work with older Android - aka, something in their system broke?
Same issue happening with Nougat, when checking out 17.3 or 17.4
Cheers
Thanks
Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
