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

Mention AUR installation and add Arch Linux dependencies to Source Build instructions #102

Merged
merged 2 commits into from
Dec 2, 2024

Conversation

Morxemplum
Copy link
Contributor

The purpose of this pull request is to improve the documentation by making the project more accessible to Arch Linux users. This is done in two ways:

  1. Mention the unofficial AUR packages by @HurricanePootis and how they can be installed
  2. Add Arch Linux equivalents to required dependencies of building the mcpelauncher components from source.

To ensure that the dependencies in Arch Linux are equivalent, I created a minimal Arch Linux distrobox image to install the dependencies and build mcpelauncher, msa, and the Qt UI completely from source. mcpelauncher and msa compiled flawlessly, but I did run into an issue related to an outdated C++ specification when compiling the Qt UI. I have raised this issue here. Bumping it up to 17 makes Qt UI compile flawlessly.

@ChristopherHX
Copy link
Member

Bumping it up to 17 makes Qt UI compile flawlessly.

This only happend in the qt6 branch, since I would assume if someone need to go with qt5 then is protobuf c++14 compatible.

I still accept a pr for the qt5 version for the time beeing before beeing killed.

After AppImage deprecation, would macOS dmg's minimum OS get's bumped and qt5 is out of order.

AUR uses qt6 for a long time

@ChristopherHX ChristopherHX merged commit 76e4f73 into minecraft-linux:main Dec 2, 2024
2 checks passed
@ChristopherHX
Copy link
Member

Thanks for writing this up 👍 merged

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants