On Mon, Aug 22, 2016 at 2:56 PM, Dave Airlie <airl...@gmail.com> wrote:
> On 23 August 2016 at 07:18, Jason Ekstrand <ja...@jlekstrand.net> wrote: > > On Mon, Aug 22, 2016 at 2:06 PM, Julien Cristau <jcris...@debian.org> > wrote: > >> > >> On Fri, Aug 19, 2016 at 09:04:14 -0700, Jason Ekstrand wrote: > >> > >> > Not providing a path allows the ICD to work on multi-arch systems but > >> > breaks it if you install anywhere other than /usr/lib. Given that > users > >> > may be installing locally in .local or similar, we probably do want to > >> > provide a filename. Distros can carry a revert of this commit if they > >> > want > >> > an intel_icd.json file without the path. > >> > > >> If a user is going to install stuff in .local, don't they have > >> LD_LIBRARY_PATH pointing there too? > > > > > > Actually, no. The loader will look for ICD files in > > .local/share/vulkan/icd.d and the ICD file will point to the right .so. > It > > should work out-of-the-box unless you either have a broken loader or > we're > > installing something wrong. > > Can we get a configure switch for this, so distros can just do that? > 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...
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev
