[Mesa-dev] [PATCH] Android: Fix vc4 build since XML changes.

Eric Anholt eric at anholt.net
Wed Jul 5 18:19:58 UTC 2017


For the automake build, -Isrc/ is implied from the gallium cflags, while
Android gallium driver builds don't get that by default.  I think it'll be
better for vc4 to have broadcom includes appear as "#include
<broadcom/subdir/whatever.h>" to make it more clear where to look in the
tree than "#include <subdir/whatever.h>" does.
---

Rob: The patch *had* changed from what I submitted -- I replaced
"intel" with "broadcom".  I wonder if maybe when you tested, you'd
just dropped it, like this patch does.

I still wish we had a public docker image we could build Android Mesa
from.  I looked at your scripts, but without access to an image, I'm
not looking to build one from scratch.

 src/broadcom/Android.genxml.mk | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/broadcom/Android.genxml.mk b/src/broadcom/Android.genxml.mk
index df44b2ec0b79..dfd00a93fbcf 100644
--- a/src/broadcom/Android.genxml.mk
+++ b/src/broadcom/Android.genxml.mk
@@ -51,7 +51,7 @@ $(intermediates)/cle/v3d_packet_v21_pack.h: $(LOCAL_PATH)/cle/v3d_packet_v21.xml
 	$(call header-gen)
 
 LOCAL_EXPORT_C_INCLUDE_DIRS := \
-	$(MESA_TOP)/src/broadcom \
+	$(MESA_TOP)/src \
 	$(intermediates)
 
 include $(MESA_COMMON_MK)
-- 
2.11.0



More information about the mesa-dev mailing list