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

lp1850729: Disable QtWayland in desktop launcher #2340

Merged
merged 3 commits into from Nov 11, 2019
Merged

lp1850729: Disable QtWayland in desktop launcher #2340

merged 3 commits into from Nov 11, 2019

Conversation

uklotzde
Copy link
Contributor

@uklotzde uklotzde commented Oct 31, 2019

Workaround for https://bugs.launchpad.net/mixxx/+bug/1850729

Mixxx has massive usability issues and a CPU utilization leak on QtWayland that is now used as the default in Fedora 31 on Wayland if QT_QPA_PLATFORM is unset. By explicitly setting it to "xcb" Qt uses XWayland as a fallback.

@uklotzde uklotzde added this to the 2.2.3 milestone Oct 31, 2019
@uklotzde uklotzde changed the base branch from master to 2.2 October 31, 2019 20:20
@daschuer
Copy link
Member

daschuer commented Nov 7, 2019

I wonder if this does not harm at all distros. Do we need to install libx11-xcb1 for that?

@Be-ing
Copy link
Contributor

Be-ing commented Nov 7, 2019

I don't think so. Fedora patched Qt to make this change: https://fedoraproject.org/wiki/Changes/Qt_Wayland_By_Default_On_Gnome

@uklotzde
Copy link
Contributor Author

uklotzde commented Nov 7, 2019

The change should work on all distros and only makes a formerly implicit assumption explicit.

My first attempt to patch the launcher failed, because I got confused by the redundant occurrences. New builds with an effective patch are available in the testing repositories. I will remove the patch with the release of 2.2.3.

http://koji.rpmfusion.org/koji/packageinfo?packageID=148

@Holzhaus
Copy link
Member

Holzhaus commented Nov 8, 2019

LGTM. Do you mind adding some info to the Wiki's troubleshooting page? Not everybody launches Mixxx via the *.desktop entry (e.g. users of Sway WM might just run the mixxx executable via dmenu/rofi/bemenu)

@uklotzde
Copy link
Contributor Author

uklotzde commented Nov 9, 2019

@Holzhaus
Copy link
Member

Merge?

@daschuer
Copy link
Member

Yes. I have just tested it on Ubuntu Xenial and it still works.
Thank you. LGTM

@daschuer daschuer merged commit 0243472 into mixxxdj:2.2 Nov 11, 2019
@Be-ing
Copy link
Contributor

Be-ing commented Nov 12, 2019

I'm okay with merging this for the stable branch, but I think we should try to get Mixxx working with the Qt Wayland platform plugin for 2.3. That may require risky changes that wouldn't be appropriate for the stable branch.

@daschuer
Copy link
Member

We should not add more requirements, that prevents releasing 2.3. IMHO it is overdue since long.

@uklotzde uklotzde deleted the 2.2_disable_qtwayland branch November 12, 2019 11:11
@Be-ing
Copy link
Contributor

Be-ing commented Nov 12, 2019

Yes, if we don't fix it by the time we release 2.3 that is okay. But I think we should try to fix it soon.

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