Re: [Mesa-dev] [PATCH] anv: add the main include directory back

2017-03-20 Thread Tapani Pälli
On 03/19/2017 11:21 PM, Emil Velikov wrote: On 19 March 2017 at 19:30, Jason Ekstrand wrote: On March 19, 2017 9:33:02 AM Grazvydas Ignotas wrote: On Sun, Mar 19, 2017 at 3:03 PM, Emil Velikov wrote: Hi Grazvydas, On 17 March 2017 at 22:05, Grazvydas Ignotas wrote: Fixes build witho

Re: [Mesa-dev] [PATCH] anv: add the main include directory back

2017-03-19 Thread Emil Velikov
On 19 March 2017 at 21:48, Grazvydas Ignotas wrote: > On Sun, Mar 19, 2017 at 11:27 PM, Emil Velikov > wrote: >> On 19 March 2017 at 16:32, Grazvydas Ignotas wrote: >> >>> With that, it's picking the earlier installed >>> /opt/xorg/include/vulkan/vulkan_intel.h instead of >>> ../../include/vulk

Re: [Mesa-dev] [PATCH] anv: add the main include directory back

2017-03-19 Thread Grazvydas Ignotas
On Sun, Mar 19, 2017 at 11:27 PM, Emil Velikov wrote: > On 19 March 2017 at 16:32, Grazvydas Ignotas wrote: > >> With that, it's picking the earlier installed >> /opt/xorg/include/vulkan/vulkan_intel.h instead of >> ../../include/vulkan/vulkan_intel.h , and that one has: >> >> #include "vulkan.h"

Re: [Mesa-dev] [PATCH] anv: add the main include directory back

2017-03-19 Thread Grazvydas Ignotas
On Sun, Mar 19, 2017 at 11:21 PM, Emil Velikov wrote: > On 19 March 2017 at 19:30, Jason Ekstrand wrote: >> On March 19, 2017 9:33:02 AM Grazvydas Ignotas wrote: >> >>> On Sun, Mar 19, 2017 at 3:03 PM, Emil Velikov >>> wrote: Hi Grazvydas, On 17 March 2017 at 22:05, Grazvyda

Re: [Mesa-dev] [PATCH] anv: add the main include directory back

2017-03-19 Thread Emil Velikov
On 19 March 2017 at 16:32, Grazvydas Ignotas wrote: > With that, it's picking the earlier installed > /opt/xorg/include/vulkan/vulkan_intel.h instead of > ../../include/vulkan/vulkan_intel.h , and that one has: > > #include "vulkan.h" > I'm wondering if that one should be although I'm not an exp

Re: [Mesa-dev] [PATCH] anv: add the main include directory back

2017-03-19 Thread Emil Velikov
On 19 March 2017 at 19:30, Jason Ekstrand wrote: > On March 19, 2017 9:33:02 AM Grazvydas Ignotas wrote: > >> On Sun, Mar 19, 2017 at 3:03 PM, Emil Velikov >> wrote: >>> >>> Hi Grazvydas, >>> >>> On 17 March 2017 at 22:05, Grazvydas Ignotas wrote: Fixes build without vulkan.h installe

Re: [Mesa-dev] [PATCH] anv: add the main include directory back

2017-03-19 Thread Grazvydas Ignotas
On Sun, Mar 19, 2017 at 9:30 PM, Jason Ekstrand wrote: > On March 19, 2017 9:33:02 AM Grazvydas Ignotas wrote: > >> On Sun, Mar 19, 2017 at 3:03 PM, Emil Velikov >> wrote: >>> >>> Hi Grazvydas, >>> >>> On 17 March 2017 at 22:05, Grazvydas Ignotas wrote: Fixes build without vulkan.h in

Re: [Mesa-dev] [PATCH] anv: add the main include directory back

2017-03-19 Thread Jason Ekstrand
On March 19, 2017 9:33:02 AM Grazvydas Ignotas wrote: On Sun, Mar 19, 2017 at 3:03 PM, Emil Velikov wrote: Hi Grazvydas, On 17 March 2017 at 22:05, Grazvydas Ignotas wrote: Fixes build without vulkan.h installed in system header locations: CC vulkan/vulkan_libvulkan_intel_la-anv_ge

Re: [Mesa-dev] [PATCH] anv: add the main include directory back

2017-03-19 Thread Grazvydas Ignotas
On Sun, Mar 19, 2017 at 3:03 PM, Emil Velikov wrote: > Hi Grazvydas, > > On 17 March 2017 at 22:05, Grazvydas Ignotas wrote: >> Fixes build without vulkan.h installed in system header locations: >> CC vulkan/vulkan_libvulkan_intel_la-anv_gem.lo >> In file included from vulkan/anv_private.

Re: [Mesa-dev] [PATCH] anv: add the main include directory back

2017-03-19 Thread Emil Velikov
Hi Grazvydas, On 17 March 2017 at 22:05, Grazvydas Ignotas wrote: > Fixes build without vulkan.h installed in system header locations: > CC vulkan/vulkan_libvulkan_intel_la-anv_gem.lo > In file included from vulkan/anv_private.h:66:0, > from vulkan/anv_gem.c:31: > /opt/xo

Re: [Mesa-dev] [PATCH] anv: add the main include directory back

2017-03-18 Thread Lionel Landwerlin
Sounds like a good idea to me. Ben had the same problem a few days ago. Acked-by: Lionel Landwerlin On 17/03/17 22:05, Grazvydas Ignotas wrote: Fixes build without vulkan.h installed in system header locations: CC vulkan/vulkan_libvulkan_intel_la-anv_gem.lo In file included from vulka

[Mesa-dev] [PATCH] anv: add the main include directory back

2017-03-17 Thread Grazvydas Ignotas
Fixes build without vulkan.h installed in system header locations: CC vulkan/vulkan_libvulkan_intel_la-anv_gem.lo In file included from vulkan/anv_private.h:66:0, from vulkan/anv_gem.c:31: /opt/xorg/include/vulkan/vulkan_intel.h:27:20: fatal error: vulkan.h: No such file or