<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Mon, Aug 22, 2016 at 2:56 PM, Dave Airlie <span dir="ltr"><<a href="mailto:airlied@gmail.com" target="_blank">airlied@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">On 23 August 2016 at 07:18, Jason Ekstrand <<a href="mailto:jason@jlekstrand.net">jason@jlekstrand.net</a>> wrote:<br>
> On Mon, Aug 22, 2016 at 2:06 PM, Julien Cristau <<a href="mailto:jcristau@debian.org">jcristau@debian.org</a>> wrote:<br>
>><br>
>> On Fri, Aug 19, 2016 at 09:04:14 -0700, Jason Ekstrand wrote:<br>
>><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<br>
>> > want<br>
>> > an intel_icd.json file without the path.<br>
>> ><br>
>> If a user is going to install stuff in .local, don't they have<br>
>> LD_LIBRARY_PATH pointing there too?<br>
><br>
><br>
> Actually, no.  The loader will look for ICD files in<br>
> .local/share/vulkan/icd.d and the ICD file will point to the right .so.  It<br>
> should work out-of-the-box unless you either have a broken loader or we're<br>
> installing something wrong.<br>
<br>
</span>Can we get a configure switch for this, so distros can just do that?<span class="HOEnZb"><font color="#888888"><br></font></span></blockquote><div><br></div><div>We could but I'm always a bit scared of adding configure switches marked "distro only" because you know what a user will do when they see one... <br></div></div><br></div></div>