[Mesa-dev] [PATCH] anv: Give the installed intel_icd.json file an absolute path

[Jason Ekstrand]

On Mon, Aug 22, 2016 at 2:56 PM, Dave Airlie <airlied at gmail.com> wrote:

> On 23 August 2016 at 07:18, Jason Ekstrand <jason at jlekstrand.net> wrote:
> > On Mon, Aug 22, 2016 at 2:06 PM, Julien Cristau <jcristau at 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...
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/mesa-dev/attachments/20160822/8bc5525a/attachment-0001.html>