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

Always run as X11 app even under Wayland #8357

Merged
merged 1 commit into from
Nov 30, 2019
Merged

Always run as X11 app even under Wayland #8357

merged 1 commit into from
Nov 30, 2019

Conversation

KAMiKAZOW
Copy link
Contributor

Dolphin doesn't have a working Wayland renderer (not even with env SDL_VIDEODRIVER=wayland dolphin-emu). By setting the QT_QPA_PLATFORM variable to "xcb" we ensure that Dolphin runs just fine in any Wayland-based desktop environment instead of displaying an error message.

@KAMiKAZOW
Copy link
Contributor Author

Ping?
It's been two months and this change is only a single line.

@Sunderland93
Copy link

@KAMiKAZOW #7452

@KAMiKAZOW
Copy link
Contributor Author

So merge this until proper Wayland support is done. This is literally just a single changed line.

@KAMiKAZOW
Copy link
Contributor Author

@Sunderland93: Why 👎?

@Techjar
Copy link
Contributor

Techjar commented Nov 23, 2019

It's just an environment variable, you can set it yourself.

@KAMiKAZOW
Copy link
Contributor Author

It's just an environment variable, you can set it yourself.

Not when there's an icon in the launcher and the user merely clicks on it. 🤦

I don't get the resistance to a very simple change that improves convenience.

What does the actual author of the Wayland back-end have to say about this, @stenzek? Will it land very soon or does it make sense to land this PR in the meantime?

@stenzek
Copy link
Contributor

stenzek commented Nov 30, 2019

Last time I checked qt defaulted to xcb even when under wayland anyway, but being explicit can't hurt..

@stenzek stenzek merged commit ba8b5be into dolphin-emu:master Nov 30, 2019
@KAMiKAZOW KAMiKAZOW deleted the patch-1 branch November 30, 2019 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

4 participants