Skip to content

[feat] [Feature] Fix wayland not being there for the appimage + Add linux build doc#1022

Merged
dail8859 merged 5 commits intodail8859:masterfrom
Twig6943:master
May 1, 2026
Merged

[feat] [Feature] Fix wayland not being there for the appimage + Add linux build doc#1022
dail8859 merged 5 commits intodail8859:masterfrom
Twig6943:master

Conversation

@Twig6943
Copy link
Copy Markdown
Contributor

solves #981 + adds linux build doc

@Twig6943
Copy link
Copy Markdown
Contributor Author

@dail8859

@Twig6943
Copy link
Copy Markdown
Contributor Author

Twig6943 commented Apr 28, 2026

Ubuntu CI seems to fail due it using github packages to setup QT. May I replace that with?

sudo apt install -y \
  cmake \
  ninja-build \
  file \
  libxkbcommon-dev \
  libxkbcommon-x11-0 \
  fuse \
  libxcb-cursor-dev \
  libcups2-dev \
  libglib2.0-0 \
  libglib2.0-dev \
  libproxy1v5 \
  libproxy-dev \
  qt6-base-dev \
  qt6-base-dev-tools \
  qt6-tools-dev \
  qt6-tools-dev-tools \
  qt6-wayland \
  qt6-wayland-dev \
  libqt6waylandclient6 \
  qml6-module-qtwayland-compositor \
  libqt6core5compat6 \
  libqt6core5compat6-dev \
  qt6-base-private-dev

@dail8859
Copy link
Copy Markdown
Owner

Could you revert the unneeded changes to the whitespace formatting and quotes please?

Ubuntu CI seems to fail due it using github packages to setup QT.

I'd prefer to keep that GitHub action as is and keep using jurplel/install-qt-action since that's what has been working so far. If there's a few extra things to manually install, or if there's another flag to pass to install-qt-action that would certainly be the best option rather than changing the action for just wayland support.

@dail8859 dail8859 added the linux Related specifically to Linux-based OSs label Apr 28, 2026
@Twig6943
Copy link
Copy Markdown
Contributor Author

think u can do that? i never use packages on purpose

@dail8859
Copy link
Copy Markdown
Owner

No clue. Just odd it works on everything else except Qt6.10

@dail8859
Copy link
Copy Markdown
Owner

I found this. Seems like the platform plugin may have got renamed for Qt6.10

HTRamsey/qgroundcontrol@fc84499

@Twig6943
Copy link
Copy Markdown
Contributor Author

I found this. Seems like the platform plugin may have got renamed for Qt6.10

HTRamsey/qgroundcontrol@fc84499

CI has matrix with qt "6.5", "6.8", "6.10" which means either needs to be rewritten or some sort of if statement which i aint exactly interested in bc ykw i dont like packages

@dail8859
Copy link
Copy Markdown
Owner

Even if it didn't use GitHub packages to install Qt you'd still have to have an if statement for the name of the plugin to include.

@Twig6943
Copy link
Copy Markdown
Contributor Author

Even if it didn't use GitHub packages to install Qt you'd still have to have an if statement for the name of the plugin to include.

Fixed.

@dail8859
Copy link
Copy Markdown
Owner

dail8859 commented May 1, 2026

Thanks! I'm assuming the AppImages are working better on Wayland now?

@Twig6943
Copy link
Copy Markdown
Contributor Author

Twig6943 commented May 1, 2026

Thanks! I'm assuming the AppImages are working better on Wayland now?

Yea there's no need for xwayland now. Please add in dark theme tho its the most essential thing this needs

[skip ci]
@dail8859 dail8859 merged commit 16a98e5 into dail8859:master May 1, 2026
1 check passed
@dail8859
Copy link
Copy Markdown
Owner

dail8859 commented May 1, 2026

Thank you for this. And thank you for adding in the documentation for building this on Linux.

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

Labels

linux Related specifically to Linux-based OSs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants