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 AppImage Packaging script #13055

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

Conversation

GT190
Copy link

@GT190 GT190 commented Sep 6, 2024

This adds a script, which builds a Dolphin AppImage. It is intended to be used with Dolphins CI, but also works when locally compiling Dolphin with these cmake flags: -DLINUX_LOCAL_DEV=true -DCMAKE_INSTALL_PREFIX=/usr -GNinja. It needs to be executed from the Build directory, which can be done with ../BuildLinuxAppImage.sh.

The script installs Dolphin into a folder called AppDir, adds the translation files into the AppDir and builds an AppImage from the AppDir folder.

Necessary for dolphin-emu/sadm#198

This saves around 16MB from the AppImage.
@OatmealDome
Copy link
Member

Just a quick note: I haven't forgotten about this PR. I will be touching the CI infrastructure in the near future to add Flatpak support, so I believe it'll be easier for me if I do this at the same time.

BuildLinuxAppImage.sh Outdated Show resolved Hide resolved
@Samueru-sama
Copy link

Hey what's holding this?

The PR looks good, I'm surprised that it even uses the right appimagetool to make the AppImage with the static appimage runtime which has no libfuse2 dependency.

@OatmealDome
Copy link
Member

Thanks for testing. This requires some CI infrastructure work before it can be merged. I'm currently working on Flatpak infrastructure and will move on to AppImage after that is done.

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