[Mesa-dev] [PATCH 11/18] anv: automake: bring back VULKAN_ENTRYPOINT_CPPFLAGS

[Alejandro Piñeiro]

On 31/05/16 16:12, Emil Velikov wrote:
> OK, so things are as expected - CPP fails to open/parse the headers.
> Why/how that happens is an interesting question... Can you try the
> following patch ? I doubt it will help much, still worth a shot
> though.
>
> diff --git a/src/intel/vulkan/Makefile.am b/src/intel/vulkan/Makefile.am
> index 37c2986..c5cb896 100644
> --- a/src/intel/vulkan/Makefile.am
> +++ b/src/intel/vulkan/Makefile.am
> @@ -125,6 +125,7 @@ nodist_EXTRA_libvulkan_intel_la_SOURCES = dummy.cpp
>  libvulkan_intel_la_SOURCES = $(VULKAN_GEM_FILES)
>
>  VULKAN_ENTRYPOINT_CPPFLAGS = \
> + -I$(top_srcdir)/include/vulkan \
>   -DVK_USE_PLATFORM_XCB_KHR \
>   -DVK_USE_PLATFORM_WAYLAND_KHR

This didn't help. In any case, the problem got solved (see below).

>
> Alternatively the following will throw everything (both parsing and
> errors from CPP) into a file parsed_header.log. Give it a try and send
> me the file/log ?
>
> diff --git a/src/intel/vulkan/Makefile.am b/src/intel/vulkan/Makefile.am
> index 37c2986..9d36b22 100644
> --- a/src/intel/vulkan/Makefile.am
> +++ b/src/intel/vulkan/Makefile.am
> @@ -129,6 +129,7 @@ VULKAN_ENTRYPOINT_CPPFLAGS = \
>        -DVK_USE_PLATFORM_WAYLAND_KHR
>
> anv_entrypoints.h : anv_entrypoints_gen.py $(vulkan_include_HEADERS)
> +       $(CPP) $(VULKAN_ENTRYPOINT_CPPFLAGS)
> $(top_srcdir)/include/vulkan/vulkan_intel.h >parsed_header.log 2>&1
>        $(AM_V_GEN)$(CPP) $(VULKAN_ENTRYPOINT_CPPFLAGS)
> $(top_srcdir)/include/vulkan/vulkan_intel.h |\
>        $(PYTHON2) $(srcdir)/anv_entrypoints_gen.py header > $@

Doing this allowed me to check that there was a missing header, so I
needed to install libwayland-dev (fwiw, Im on a system reinstalled from
scratch recently). So I guess that right now the problem is that
configure should ask for this dependency. I still had problems, but
those got solved with your fix to bug 96285.

Thanks for your help

BR