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
firefox-wrapper: rename gdkWayland->forceWayland; always use libglvnd #73365
Conversation
7d702a2
to
98c2db7
Compare
FYI |
@adisbladis Does that just make this PR redundant then? I don't fully understand GL, let alone GL in Nix, or propagated inputs, embarrassingly. If so, I'll pull those commits and test and can close this out. Thanks for the heads up. |
IIRC, compiled propagated build inputs do not work well with cross builds |
It's been a couple of months and I can confirm that this commit is still required to get WebGL (https://get.webgl.org) working with Firefox stable on Or if there's a better way to solve this, I can update the PR? Thanks! EDIT: This is the output without
|
98c2db7
to
3c71ec7
Compare
(Also, I'm new to |
@colemickens I've just visited the website you mentioned. It reports 'Your browser supports WebGL`, and I don't see any of those errors in the console. This machine isn't using iris though, so I'm betting somehow that has something to do with. |
Strange. I disabled Iris and restarted Sway and starting Firefox from a terminal still absolutely fills my console with "Failed to load EGL library 3!" etc, and the same error specifically on webgl.org:
To be clear:
|
3c71ec7
to
6330014
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/hardware-acceleration-on-firefox/7947/9 |
6330014
to
4cbcb26
Compare
4cbcb26
to
68c68ff
Compare
This pull request has been mentioned on NixOS Discourse. There might be relevant details there: https://discourse.nixos.org/t/hardware-acceleration-on-firefox/7947/11 |
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.
diff LGTM
didn't test though
Motivation for this change
Without this change, when starting Firefox with WebRender enabled, the default standard out/err is filled with errors.
Further, this cleans up some old remnants of what looks like early Firefox Wayland support that is no longer needed. I'm not even sure
libglvnd
needs to be optional.Note: I'm not sure if I need to do this differently, or make more changes, for backward compat purposes?
A somewhat clipped log when starting Firefox (with WebRender enabled) without my change:
Note this one:
Crash Annotation GraphicsCriticalError: |[0][GFX1-]: Failed to load EGL library: FEATURE_FAILURE_EGL_LOAD_3 (t=0.226486) |[1][GFX1-]: Failed to load EGL library: FEATURE_FAILURE_EGL_LOAD_3 (t=0.226627) |[2][GFX1-]: Failed to load EGL library: FEATURE_FAILURE_EGL_LOAD_3 (t=0.577772) |[3][GFX1-]: Failed to get shared GL context (t=0.577814) |[4][GFX1-]: Failed to load EGL library 3! (t=0.577919) |[5][GFX1-]: Failed GL context creation for WebRender: 0 (t=0.577936) |[6][GFX1-]: Failed to connect WebRenderBridgeChild. (t=0.578027) |[7][GFX1-]: Failed to load EGL library 3! (t=0.578314)
.Afterward, the complete, non-clipped log is:
More verification that this is doing something good is still a wip.
Things done
sandbox
innix.conf
on non-NixOS linux)nix-shell -p nix-review --run "nix-review wip"
./result/bin/
)nix path-info -S
before and after)Notify maintainers
cc @