<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Aug 19, 2016 at 11:25 PM, Jason Ekstrand <span dir="ltr"><<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="HOEnZb"><div class="h5"><p dir="ltr"></p>
<p dir="ltr">On Aug 19, 2016 20:08, "Chad Versace" <<a href="mailto:chad@kiwitree.net" target="_blank">chad@kiwitree.net</a>> wrote:<br>
><br>
> On Fri 19 Aug 2016, Jason Ekstrand wrote:<br>
> > Not providing a path allows the ICD to work on multi-arch systems but<br>
> > breaks it if you install anywhere other than /usr/lib.  Given that users<br>
> > may be installing locally in .local or similar, we probably do want to<br>
> > provide a filename.  Distros can carry a revert of this commit if they want<br>
> > an intel_icd.json file without the path.<br>
> ><br>
> > Signed-off-by: Jason Ekstrand <<a href="mailto:jason@jlekstrand.net" target="_blank">jason@jlekstrand.net</a>><br>
> > Cc: Mark Janes <<a href="mailto:mark.a.janes@intel.com" target="_blank">mark.a.janes@intel.com</a>><br>
> > ---<br>
> >  src/intel/vulkan/.gitignore                            | 1 +<br>
> >  src/intel/vulkan/Makefile.am                           | 7 ++++++-<br>
> >  src/intel/vulkan/{intel_icd.<wbr>json => <a href="http://intel_icd.json.in" target="_blank">intel_icd.json.in</a>} | 2 +-<br>
> >  3 files changed, 8 insertions(+), 2 deletions(-)<br>
> >  rename src/intel/vulkan/{intel_icd.<wbr>json => <a href="http://intel_icd.json.in" target="_blank">intel_icd.json.in</a>} (59%)<br>
><br>
> Yes please.<br>
> Reviewed-by: Chad Versace <<a href="mailto:chad@kiwitree.net" target="_blank">chad@kiwitree.net</a>></p>
</div></div><p dir="ltr">Feel free to push. I'm traveling.<br>
</p>
</blockquote></div>Found airport wifi and pushed.  Thanks for the reviews!<br></div></div>