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

[Feature Request]: Flatpak Support. #183

Open
3 tasks done
Atkoset opened this issue Jun 5, 2022 · 17 comments
Open
3 tasks done

[Feature Request]: Flatpak Support. #183

Atkoset opened this issue Jun 5, 2022 · 17 comments
Labels
accepted This issue has been accepted enhancement New feature or request

Comments

@Atkoset
Copy link

Atkoset commented Jun 5, 2022

Preflight checklist

  • I have searched the issue tracker for a bug report that matches the one I want to file, without success.
  • I have searched the features list for this feature, and I couldn't find it.
  • I have checked that I'm using the latest stable version of the app.

Problem Description

Hello, I would like to see the Flatpak packaging to streamline updates with the package manager so everything can be updated all at once when doing system updates on the Steamdeck. Since many Linux distro's come with the Flatpak package manager they will better and more easily integrate with SteamOS other Linux distro's. So the main features are:

  1. Easy to develop.
  2. Easy to maintain and install on the users end.
  3. Easy to update and manage.
  4. Very stable
  5. Consistent environments

Proposed Solution

My proposed solution would be to slowly transition over to Flatpak and have that as another option for Linux users.

@Atkoset Atkoset added the enhancement New feature or request label Jun 5, 2022
@ThaUnknown
Copy link
Owner

I wont pursue this, but I'll accept a PR

@Atkoset
Copy link
Author

Atkoset commented Jun 6, 2022

Unfortunately due to my personal circumstances I won't be able to do this, so I recommend keeping this issue open for in the future as a possibility or for someone looking to contribute to the project by accepting tickets.

@ThaUnknown ThaUnknown added the accepted This issue has been accepted label Jun 12, 2022
@sambuca1g
Copy link
Contributor

Hey there, @ThaUnknown! I started working on the flatpak, but got halted at an error which involves a few problems with the .desktop file. It might be worth another issue, but let's see what you think about it.

  • The "Video" category requires another category called "AudioVideo" as well.
  • The "Anime" category is an unregistered value, so it should be renamed to "X-Anime" or completely removed.
  • At "MimeType" "x-scheme-handler/miru" is included twice. Though not an error, but a warning.

Hopefully, fixing these minor problems will also grant the way for the flatpak too.

@Muril-o
Copy link
Collaborator

Muril-o commented Aug 19, 2022

You can straight up remove Anime and add AudioVideo to match the freedesktop specifications on registered categories, theres no real threat to it as it won't break anything.

One of the MimeTypes can be removed in package.json if protocol is later tested on a fresh install and works for both .AppImage (test this first) and .deb (test this second). I remember electron having ambiguous documentation on linux mimetypes so I added both just in case.

I'll reopen this issue since you want to pursue this.

@Muril-o Muril-o reopened this Aug 19, 2022
@ThaUnknown
Copy link
Owner

Hey there, @ThaUnknown! I started working on the flatpak, but got halted at an error which involves a few problems with the .desktop file. It might be worth another issue, but let's see what you think about it.

  • The "Video" category requires another category called "AudioVideo" as well.
  • The "Anime" category is an unregistered value, so it should be renamed to "X-Anime" or completely removed.
  • At "MimeType" "x-scheme-handler/miru" is included twice. Though not an error, but a warning.

Hopefully, fixing these minor problems will also grant the way for the flatpak too.

PR this an I'll merge it

@sambuca1g
Copy link
Contributor

PR this an I'll merge it

Alright, so it is really not needed to now because I can just source a modified .desktop file. If you want to change it though, it's up to you.
Anyways, I got it working using the AppImage and it is performing pretty well. Are there any specific filesystem permissions that I should add, like access to Downloads folder? I kept it really minimal.

@ThaUnknown
Copy link
Owner

just... PR the changes

not that I'd know, as you can see I don't use/test linux

@sambuca1g
Copy link
Contributor

just... PR the changes

not that I'd know, as you can see I don't use/test linux

Did a PR.

@sambuca1g
Copy link
Contributor

sambuca1g commented Aug 28, 2022

Okay. I set up a repo, feel free to check it out. I'll have to double check the permissions though.
Now the question is: should we post it to Flathub? It'll definitely bring a broader userbase. However, I don't know if they consider using this project as piracy and if they would allow it.

@ThaUnknown
Copy link
Owner

Okay. I set up a repo, feel free to check it out. I'll have to double check the permissions though. Now the question is: should we post it to Flathub? It'll definitely bring a broader userbase. However, I don't know if they consider using this project as piracy and if they would allow it.

... PR it

@Atkoset
Copy link
Author

Atkoset commented Sep 20, 2022

Okay. I set up a repo, feel free to check it out. I'll have to double check the permissions though. Now the question is: should we post it to Flathub? It'll definitely bring a broader userbase. However, I don't know if they consider using this project as piracy and if they would allow it.

... PR it

Considering that qbittorent and one other similar app to Miru is offered there It should be safe to post it on Flathub.

@TomatDividedBy0
Copy link

I was able to successfully install using the repository built in the mentioned issue:
flathub/flathub#4139 (comment)

Also fixes a problem I was having signing in which was present with the AppImage.

@sambuca1g
Copy link
Contributor

It's available. With the merge of #271, I think we can close this issue.

@Jeremi360
Copy link

For me flatpak don't want to launch, I get this when I try to run it:

09:26:20.353 › APPIMAGE env is not defined, current application is not an AppImage
LaunchProcess: failed to execvp:
xdg-settings
09:26:20.360 › checkForUpdatesAndNotify called, downloadPromise is null
[3:0913/092620.361863:ERROR:ozone_platform_x11.cc(240)] Missing X server or $DISPLAY
[3:0913/092620.361901:ERROR:env.cc(255)] The platform failed to initialize.  Exiting.
The futex facility returned an unexpected error code.
/app/bin/miru-run: linia 3:     3 Przerwane               zypak-wrapper /app/bin/miru/miru "$@"

@cassiofb-dev
Copy link

Loved the flatpak version, runs fine for me the only issue I found is the error message when I quit the application:

image

@Victor239
Copy link

There's two things missing from this Flathub support:

  • There is no automated changelog for new releases
  • It's "community built" - it would gain more trust if it's becomes a verified official package

@Temanor
Copy link

Temanor commented May 27, 2024

Doesn't look like the flatpak has been maintained in 4 months now. It is still on version 4.5.10.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
accepted This issue has been accepted enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

9 participants