Created attachment 225508 [details]
1.5.8 reverted two commits that caused regressions under certain conditions for Xorg, but regressed Wayland-only environments where libGL does not exist and OpenGL is provided by libOpenGL instead.
Fix Wayland-only environments by making a reverted patch conditional when X11=off.
The current 1.5.8 in ports breaks at least www/webkit2-gtk3 for libglvnd and Wayland-only:
Couldn't open libGL.so.1: Shared object "libGL.so.1" not found
Has this patch been submitted and merged upstream?
I think I found the commit:
If that's the case, the patch is already included in the release.
I might be confused. There is, however, also this commit
That seem to revert the change in 1.5.8.
Ok, I figured it out.
The patches were reverted upstream in 1.5.8 because they caused regressions. Before including this patch again, please confer with upstream on how to best handle the situation. I would very much prefer not to add this as a local patch as possible, especially since it has both been committed and reverted upstream.
(In reply to Niclas Zeising from comment #4)
OK. There is https://github.com/anholt/libepoxy/pull/232 though it doesn't get much activity. Thanks for reply, I'll leave bug open so the libGL message is searchable.
This is a big can of worms. The right place to get this fixed is upstream since there's several issues noted in the linked info from upstream. Given the issues it caused upstream, we've opted to not include this patch. Once upstream sorts it out we'll get it on the next update.
Talked with upstream, https://github.com/anholt/libepoxy/pull/257 will fix this. I'll reopen this and use it as a self reminder.
x11, koobs, are you ok with me pulling in this patch to our tree for testing? We can report success or failure to upstream to help get it in a release. This one looks low risk to me because it prefers the existing setup (GLX) and therefore the increased risk is heavily weighted to the !X11 option.
(In reply to Kevin Bowling from comment #8)
To clarify, https://github.com/anholt/libepoxy/pull/257, not the patch that was backed out.
(In reply to Kevin Bowling from comment #9)
Changes that fix build issues (build, configure, package, etc) and are QA'd are:
Approved by: portmgr (blanket: <target> fix)
MFH: <$branch|No) (<reason>)
Committers may self-assign and resolve at any time under those conditions
Since the original patch suggested here is a patch that was committed and then backed out, and since there is an ongoing discussion upstream on how to solve this, I ask, with my x11@ hat on, to please hold off on committing the patch in this PR or the suggested patch from upstream, until the upstream discussion reaches a solution and a patch has been merged upstream.