<p dir="ltr"></p>
<p dir="ltr">On Aug 19, 2016 20:08, "Chad Versace" <<a href="mailto:chad@kiwitree.net">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">jason@jlekstrand.net</a>><br>
> > Cc: Mark Janes <<a href="mailto:mark.a.janes@intel.com">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.json => <a href="http://intel_icd.json.in">intel_icd.json.in</a>} | 2 +-<br>
> >  3 files changed, 8 insertions(+), 2 deletions(-)<br>
> >  rename src/intel/vulkan/{intel_icd.json => <a href="http://intel_icd.json.in">intel_icd.json.in</a>} (59%)<br>
><br>
> Yes please.<br>
> Reviewed-by: Chad Versace <<a href="mailto:chad@kiwitree.net">chad@kiwitree.net</a>></p>
<p dir="ltr">Feel free to push. I'm traveling.<br>
</p>