Hi Mauro,

Thanks for the series. I'll pull 2&3 in a second - there's a minor
suggestion in this patch.

On 3 August 2017 at 01:55, Mauro Rossi <issor.or...@gmail.com> wrote:
> Android build changes to avoid the following building error:
>
> target  C: libmesa_pipe_radeonsi <= 
> external/mesa/src/gallium/drivers/radeonsi/si_pipe.c
> ...
> In file included from external/mesa/src/gallium/drivers/radeonsi/si_pipe.c:38:
> external/mesa/src/compiler/nir/nir.h:48:10: fatal error: 'nir_opcodes.h' file 
> not found
> #include "nir_opcodes.h"
>          ^
> 1 error generated.
>
> Fixes: da62a31c5b "radeonsi: add nir include paths"
> ---
>  src/gallium/drivers/radeonsi/Android.mk | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
>
> diff --git a/src/gallium/drivers/radeonsi/Android.mk 
> b/src/gallium/drivers/radeonsi/Android.mk
> index 6fff91f6f7..452bba3af8 100644
> --- a/src/gallium/drivers/radeonsi/Android.mk
> +++ b/src/gallium/drivers/radeonsi/Android.mk
> @@ -36,7 +36,8 @@ LOCAL_MODULE_CLASS := STATIC_LIBRARIES
>
>  LOCAL_C_INCLUDES := \
>         $(MESA_TOP)/src/amd/common \
> -       $(call 
> generated-sources-dir-for,STATIC_LIBRARIES,libmesa_amd_common,,)/common
> +       $(call 
> generated-sources-dir-for,STATIC_LIBRARIES,libmesa_amd_common,,)/common \
> +       $(call generated-sources-dir-for,STATIC_LIBRARIES,libmesa_nir,,)/nir
The more robust solution is to add LOCAL_EXPORT_C_INCLUDE_DIRS for libmesa_nir.

With that in place we can drop the existing four instances of the
$(call generated-sources-dir-for... libmesa_nir... from the codebase.

-Emil
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to