1767231 - Snap: severe regression: MOZ_ENABLE_WAYLAND=1 does no longer work, fallback to SW WR

Status: VERIFIED FIXED

(Core :: Widget: Gtk, defect)

Description

[Darkspirit]

Debian Testing, Gnome Wayland, Intel

$ MOZ_ENABLE_WAYLAND=1 snap run firefox
Gtk-Message: 10:30:24.009: Failed to load module "canberra-gtk-module"
Gtk-Message: 10:30:24.011: Failed to load module "canberra-gtk-module"
[GFX1-]: glxtest: Could not connect to wayland socket
Gtk-Message: 10:30:26.399: Failed to load module "canberra-gtk-module"
Gtk-Message: 10:30:26.400: Failed to load module "canberra-gtk-module"
[GFX1-]: glxtest: Could not connect to wayland socket

$ DISABLE_WAYLAND=0 snap run firefox
Gtk-Message: 10:32:30.827: Failed to load module "canberra-gtk-module"
Gtk-Message: 10:32:30.829: Failed to load module "canberra-gtk-module"
ATTENTION: default value of option mesa_glthread overridden by environment.
ATTENTION: default value of option mesa_glthread overridden by environment.

$ DISABLE_WAYLAND=0 MOZ_ENABLE_WAYLAND=1 snap run firefox
Gtk-Message: 10:33:33.541: Failed to load module "canberra-gtk-module"
Gtk-Message: 10:33:33.542: Failed to load module "canberra-gtk-module"
[GFX1-]: glxtest: Could not connect to wayland socket

Assumingly regressed by https://git.launchpad.net/~mozilla-snaps/+git/firefox-snap/commit/?id=4f3ad5cdc390fedab5db6e2d9a062a2c3bd88474 (bug 1725245 comment 19).

Comment 1

[Darkspirit]

Not having Wayland on Wayland (re-)introduces problems: bug 1766849, bug 1765020, bug 1748921, bug 1744389, bug 1735929, bug 1736245

Comment 2

[Olivier Tilloy]

You're right, that change unfortunately made it impossible to force the native use of Wayland. I'm looking into a less invasive approach, that would still default to XWayland (until it is deemed ready for native Wayland), but allow forcing native Wayland for testing purposes.

Comment 3

[Darkspirit]

Snap Firefox Nightly and Beta are unaffected.

Comment 4

[Olivier Tilloy]

This commit should do it. If MOZ_ENABLE_WAYLAND is unset, set it to 0. Otherwise, respect the value being passed.

Comment 5

[Ryan VanderMeulen [:RyanVM]]

IIUC, this is fixed now Olivier?

Comment 6

[Olivier Tilloy]

Yes, this is now fixed with version 100.0-2 in the stable channel.

Comment 7

[Darkspirit]

Verified fixed, thanks.

xwayland: $ snap run firefox
wayland: $ MOZ_ENABLE_WAYLAND=1 snap run firefox
xwayland: $ MOZ_ENABLE_WAYLAND=0 snap run firefox