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

Add Wayland support in AppImages #1663

Merged
merged 1 commit into from
Oct 10, 2021

Conversation

J5lx
Copy link
Member

@J5lx J5lx commented Sep 22, 2021

I've been using Wayland as my daily driver for a while now and it seems to work well enough nowadays. A build for testing is here.

I've been using Wayland as my daily driver for a while now and it seems to work
well enough nowadays.
@MrStevns
Copy link
Member

MrStevns commented Oct 6, 2021

LGTM, do you need others to test or can I merge the PR?

@J5lx
Copy link
Member Author

J5lx commented Oct 6, 2021

I would prefer if someone who also uses Wayland could give the build I made a quick try. Since this PR is only about packaging (with all the actual heavy lifting handled by Qt), I wouldn’t consider a comprehensive test to be necessary though. Just a quick check to make sure everything is wired up correctly should be enough.

@scribblemaniac
Copy link
Member

Tested under X11, Wayland with XWayland, and Wayland with Wayland (QT_QPA_PLATFORM=wayland). In all cases it seems to work fine, ready to merge 👍

@J5lx
Copy link
Member Author

J5lx commented Oct 10, 2021

Great, thanks for giving it a try!

@J5lx J5lx merged commit 930fb41 into pencil2d:master Oct 10, 2021
@J5lx J5lx deleted the enhancements/appimage-wayland branch October 10, 2021 10:29
@MrStevns MrStevns added this to the v0.6.7 milestone Jul 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: No status
Development

Successfully merging this pull request may close these issues.

3 participants