On Mon, Mar 20, 2017 at 5:18 PM, Emil Velikov <[email protected]> wrote: > From: Emil Velikov <[email protected]> > > At the moment, we would honour any system headers - vulkan_intel.h in > particular over the ones in-tree. > > Thus, if one does inclemental build of mesa, without the vulkan.h > already installed (or at least not in the same directory as > vulkan_intel.h) the build will fail. > > In the future we might want to upstream the vulkan_intel.h within > vulkan.h, but in either case the more robust thing is to rely on our own > copy. > > Cc: Grazvydas Ignotas <[email protected]> > Fixes: ee8044fd "intel/vulkan: Get rid of recursive make" > Suggested-by: Jason Ekstrand <[email protected]> > Reported-by: Grazvydas Ignotas <[email protected]> > Signed-off-by: Emil Velikov <[email protected]> > --- > Grazvydas, since I butchered if while trying to explain myself I went > ahead and addressed the problem. Please double-check on your end.
This works so I'm happy enough, but wouldn't it be better to just swap $(AM_CPPFLAGS) and $(LIBDRM_CFLAGS) (which also works for me)? As mentioned, $(AM_CPPFLAGS) contains -I<install_include_path> after mesa includes, which means installed includes will get priority over -I$(top_srcdir)/src and friends from VULKAN_CPPFLAGS. Gražvydas _______________________________________________ mesa-dev mailing list [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
