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 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev