<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Sat, Jun 4, 2016 at 1:55 PM, Dave Airlie <span dir="ltr"><<a href="mailto:airlied@gmail.com" target="_blank">airlied@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">From: Dave Airlie <<a href="mailto:airlied@redhat.com">airlied@redhat.com</a>><br>
<br>
The fix in:<br>
anv: let anv_entrypoints_gen.py generate proper Wayland/Xcb guards<br>
<br>
breaks things if wayland headers aren't installed.<br>
<br>
Separate things out properly to avoid that problem.<br>
<br>
Reported-by: Arjan van de Ven<br>
Signed-off-by: Dave Airlie <<a href="mailto:airlied@redhat.com">airlied@redhat.com</a>><br>
---<br>
 src/intel/vulkan/Makefile.am | 14 +++++++++++---<br>
 1 file changed, 11 insertions(+), 3 deletions(-)<br>
<br>
diff --git a/src/intel/vulkan/Makefile.am b/src/intel/vulkan/Makefile.am<br>
index 3294fa4..0efcc4d 100644<br>
--- a/src/intel/vulkan/Makefile.am<br>
+++ b/src/intel/vulkan/Makefile.am<br>
@@ -124,11 +124,19 @@ VULKAN_LIB_DEPS += \<br>
 nodist_EXTRA_libvulkan_intel_la_SOURCES = dummy.cpp<br>
 libvulkan_intel_la_SOURCES = $(VULKAN_GEM_FILES)<br>
<br>
-VULKAN_ENTRYPOINT_CPPFLAGS = \<br>
-       $(XCB_DRI3_CFLAGS) \<br>
+VULKAN_ENTRYPOINT_CPPFLAGS =<br>
+<br>
+if HAVE_PLATFORM_WAYLAND<br>
+VULKAN_ENTRYPOINT_CPPFLAGS += \<br>
        $(WAYLAND_CFLAGS) \<br>
-       -DVK_USE_PLATFORM_XCB_KHR \<br>
        -DVK_USE_PLATFORM_WAYLAND_KHR<br>
+endif<br>
+<br>
+if HAVE_PLATFORM_X11<br>
+VULKAN_ENTRYPOINT_CPPFLAGS += \<br>
+       $(XCB_DRI3_CFLAGS) \<br>
+       -DVK_USE_PLATFORM_XCB_KHR<br>
+endif<br></blockquote><div><br></div><div>We should probably put these in the "if HAVE_PLATFORM_FOO" sections above rather than repeating it.  Other than that, this LGTM.  I think long-term, we just need to make the generator less stupid but this does the trick for now.  With things in the proper sections,<br><br></div><div>Reviewed-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>><br><br></div><div>The sad part is that we've now come full-circle.  This is more-or-less what we had when I first merged in the driver.  :-(<br><br></div><div>--Jason<br></div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
 anv_entrypoints.h : anv_entrypoints_gen.py $(vulkan_include_HEADERS)<br>
        $(AM_V_GEN)$(CPP) $(VULKAN_ENTRYPOINT_CPPFLAGS) $(top_srcdir)/include/vulkan/vulkan_intel.h |\<br>
<span class="HOEnZb"><font color="#888888">--<br>
2.5.5<br>
<br>
_______________________________________________<br>
mesa-dev mailing list<br>
<a href="mailto:mesa-dev@lists.freedesktop.org">mesa-dev@lists.freedesktop.org</a><br>
<a href="https://lists.freedesktop.org/mailman/listinfo/mesa-dev" rel="noreferrer" target="_blank">https://lists.freedesktop.org/mailman/listinfo/mesa-dev</a><br>
</font></span></blockquote></div><br></div></div>