Not an expert on the android build system, I have a question below.
Otherwise :

Acked-by: Lionel Landwerlin <lionel.g.landwer...@intel.com>

Thanks!

On 29/06/17 01:46, Mauro Rossi wrote:
In addition to Rob Herring "Android: i965: remove libdrm_intel dependency",
we can drop libdrm_intel dependency in anv for Android.

Please check if libdrm has to stay as shared dependency and drop this comment 
line.

Fixes: 7dd20bc ("anv/i965: drop libdrm_intel dependency completely")
---
  src/intel/Android.vulkan.mk          |  3 ++-
  1 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/intel/Android.vulkan.mk b/src/intel/Android.vulkan.mk
index 095a75bbd4..8f8366aa7f 100644
--- a/src/intel/Android.vulkan.mk
+++ b/src/intel/Android.vulkan.mk
@@ -33,6 +33,7 @@ VULKAN_COMMON_INCLUDES := \
        $(MESA_TOP)/src/vulkan/wsi \
        $(MESA_TOP)/src/vulkan/util \
        $(MESA_TOP)/src/intel \
+       $(MESA_TOP)/src/intel/drm \
        $(MESA_TOP)/src/intel/vulkan
# libmesa_anv_entrypoints with header and dummy.c
@@ -93,7 +94,7 @@ LOCAL_C_INCLUDES := $(ANV_INCLUDES)
LOCAL_WHOLE_STATIC_LIBRARIES := libmesa_anv_entrypoints libmesa_genxml -LOCAL_SHARED_LIBRARIES := libdrm_intel
+LOCAL_SHARED_LIBRARIES := libdrm

Wasn't libdrm already linked against? As far as I can tell it was with autotools.

include $(MESA_COMMON_MK)
  include $(BUILD_STATIC_LIBRARY)


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

Reply via email to