<div dir="auto"><div><div class="gmail_extra"><div class="gmail_quote">On Jan 13, 2017 7:22 AM, "Emil Velikov" <<a href="mailto:emil.l.velikov@gmail.com">emil.l.velikov@gmail.com</a>> wrote:<br type="attribution"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On 13 January 2017 at 15:02, Lionel Landwerlin<br>
<div class="elided-text"><<a href="mailto:lionel.g.landwerlin@intel.com">lionel.g.landwerlin@intel.com</a><wbr>> wrote:<br>
> On 13/01/17 14:31, Emil Velikov wrote:<br>
>><br>
>> On 13 January 2017 at 12:00, Lionel Landwerlin<br>
>> <<a href="mailto:lionel.g.landwerlin@intel.com">lionel.g.landwerlin@intel.com</a><wbr>> wrote:<br>
>>><br>
>>> v2: rework entry point iteration (Jason)<br>
>>>      cleanup unused imports<br>
>>><br>
>>> Signed-off-by: Lionel Landwerlin <<a href="mailto:lionel.g.landwerlin@intel.com">lionel.g.landwerlin@intel.com</a><wbr>><br>
>>> ---<br>
>>>   src/intel/vulkan/Makefile.am            |  15 ++--<br>
>>>   src/intel/vulkan/anv_<wbr>entrypoints_gen.py | 126<br>
>>> ++++++++++++++++--------------<wbr>--<br>
>>>   2 files changed, 71 insertions(+), 70 deletions(-)<br>
>>><br>
>>> diff --git a/src/intel/vulkan/Makefile.am b/src/intel/vulkan/Makefile.am<br>
>>> index df7645fb13..d32b57f267 100644<br>
>>> --- a/src/intel/vulkan/Makefile.am<br>
>>> +++ b/src/intel/vulkan/Makefile.am<br>
>>> @@ -23,11 +23,6 @@ include Makefile.sources<br>
>>><br>
>>>   vulkan_includedir = $(includedir)/vulkan<br>
>>><br>
>>> -vulkan_include_HEADERS = \<br>
>>> -       $(top_srcdir)/include/vulkan/<wbr>vk_platform.h \<br>
>>> -       $(top_srcdir)/include/vulkan/<wbr>vulkan.h \<br>
>>> -       $(top_srcdir)/include/vulkan/<wbr>vulkan_intel.h<br>
>>> -<br>
>><br>
>> I think that in the long term we might want to remove the local<br>
>> headers/xml files and use the Khronos ones directly. Distros, Arch at<br>
>> least, already ship them in separate package.<br></div></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">So does Fedora.  They get installed as part of vulkan-devel which is the dev package for the loader.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="elided-text">
>> Until then we really want to keep the above hunk. vulkan_intel.h must<br>
>> always be there - Mesa is the canonical source of it, afaict.<br></div></blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">Yes, we want to keep the vulkan_intel.h part but should probably drop the rest eventually.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_extra"><div class="gmail_quote"><blockquote class="quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="elided-text">
> But they're not a dependency to build the entry point files anymore with<br>
> these 2 patches.<br>
> So why have them as a dependency?<br>
><br>
</div>You've already handled dependency tracking nicely in the now missing hunk.<br>
The above 4-5 lines install the headers.</blockquote></div></div></div><div dir="auto"><br></div><div dir="auto">Thanks for catching that.  I think we do want to drop installing everything except vulkan_intel.h but it should be a separate patch.</div><div dir="auto"></div></div>