ubuntu / snapcraft-desktop-helpers

Various launchers for snapcraft wiki parts
MIT License
73 stars 60 forks source link

Prepend GL paths rather than appending #212

Closed Saviq closed 4 years ago

Saviq commented 4 years ago

The problem with __EGL_VENDOR_LIBRARY_DIRS is that the first one that has any configuration (e.g. 50_mesa.json) will be picked up, even if further dirs have higher priority ones (e.g. 10_nvidia.json.

snapd only puts stuff in /var/lib/snapd/lib/gl{,vnd} on systems with Nvidia, so snaps that use GL ship the Mesa stack "as a fallback". But it's the system one that should be picked up first.

More context: https://github.com/NVIDIA/libglvnd/blob/master/src/EGL/icd_enumeration.md

Saviq commented 4 years ago

CC: @zyga @RAOF @AlanGriffiths

kenvandine commented 4 years ago

This makes sense to me, thanks!

RAOF commented 4 years ago

This looks like the right thing, yes.

kenvandine commented 4 years ago

Are you going to propose these changes to the desktop extensions in snapcraft?

Saviq commented 4 years ago

Are you going to propose these changes to the desktop extensions in snapcraft?

Now I am…