New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
epoxy: 1.3.1 -> 1.5.0, add mesa to libepoxy runpath as fallback #35369
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks! Looks fine. But please rebase and target staging for this PR as it's a mass rebuild.
- libepoxy dlopen()s libEGL / libGL but didn't have mesa in its runpath -> error unless lib is already open or in LD_LIBRARY_PATH - change dependency from mesa (should be avoided) to mesa_nonglu
just out of curiousity: what triggered three review requests? is that intended behavior or did I do sth wrong? Edit: some of these autogenerated labels look wrong too... |
@xeji that's just GitHub that has issues with the automatic review requests feature in conjunction with changes base. It happens a lot, unfortunately. |
@GrahamcOfBorg eval |
just noticed v1.3.1 was over 2.5 years old. time to update... |
This breaks
|
oops... will make a fix limiting the patchelf to linux. |
Missing runtime dependency: libepoxy dlopen()s libEGL / libGL but didn't have mesa in its runpath
-> error unless lib is already open or in LD_LIBRARY_PATH
change expoxy dependency from mesa (should be avoided) to mesa_noglu
Motivation for this change
libepoxy dlopen()s libs from mesa, so mesa should be in its runpath as a fallback.
This change has no effect if the lib is already open or found in LD_LIBRARY_PATH, which is the case in most applications (or else this bug would have surfaced earlier...).
But it will retrigger a number of rebuilds.
I ran across this bug when trying to run qemu built with opengl/virglrenderer support.
Things done
build-use-sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)