Navigation Menu

Skip to content
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

Provide libwpe for webkitgtk in Wayland #103728

Closed
wants to merge 3 commits into from

Conversation

matthewbauer
Copy link
Member

When running in a Wayland environment, webkitgtk needs the libwpe backend to implement accelerated compositing on Wayland. Without this, you can get errors like:

Cannot get default EGL display: EGL_BAD_PARAMETER

Webkitgtk is supposed to gracefully disable AC when this happens, but this doesn't appear to be working for me. You can manually disable AC with WEBKIT_DISABLE_COMPOSITING_MODE=1.

See https://gitweb.gentoo.org/repo/gentoo.git/tree/net-libs/webkit-gtk/webkit-gtk-2.28.4.ebuild#n222 for some idea of how Gentoo handles this. In nixpkgs, I think it's a good idea to just provide libwpe everywhere, regardless of whether the user has configured "wayland" or not.

Other discussions:

@nixos-discourse
Copy link

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:

https://discourse.nixos.org/t/install-nixos-on-laptop-im-failing-hard/6955/5

Copy link
Contributor

@jtojnar jtojnar left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Weird, I thought we had already wpewebkit PR but cannot find it.

pkgs/development/libraries/libwpe/default.nix Outdated Show resolved Hide resolved
@matthewbauer
Copy link
Member Author

matthewbauer commented Nov 13, 2020

Weird, I thought we had already wpewebkit PR but cannot find it.

Ah I had opened it in #96140 but forgot about it! Anyway this should be find to merge first, especially if it resolves the epiphany on wayland issue.

description = "General-purpose library for WPE WebKit";
license = licenses.bsd2;
homepage = "https://wpewebkit.org";
maintainers = [ maintainers.matthewbauer ];
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
maintainers = [ maintainers.matthewbauer ];
maintainers = with maintainers; [ matthewbauer ];

description = "Freedesktop.org backend for WPE WebKit";
license = licenses.bsd2;
homepage = "https://wpewebkit.org";
maintainers = [ maintainers.matthewbauer ];
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
maintainers = [ maintainers.matthewbauer ];
maintainers = with maintainers; [ matthewbauer ];

pkgs/development/libraries/libwpe/fdo.nix Outdated Show resolved Hide resolved
pkgs/development/libraries/libwpe/fdo.nix Show resolved Hide resolved
@matthewbauer
Copy link
Member Author

Updated

@matthewbauer
Copy link
Member Author

@GrahamcOfBorg build epiphany

];

nativeBuildInputs = [
pkgconfig
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

eval error

@SuperSandro2000
Copy link
Member

@matthewbauer Can you fix the eval error?

@matthewbauer
Copy link
Member Author

Closing in favor of #96140

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants