-
Notifications
You must be signed in to change notification settings - Fork 212
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
Provide a Flatpak package. #309
Comments
Is this a religious question? Do you have any arguments? |
@Skif-off, my rationale for this proposition is that deployment of Additionally, because I utilize other I know not how this proposition is able to be religious, and I doubt that discussion of that attribute shall affect implementation of this. I consequently intend not to ascertain how. Regardless, I intend not to cause argumentation. |
This is controversial opinion. In Debian (and based on it): first version is 0.5.7 (25 Sep 2013), so people suffer 8 years... Or not?
Holy Wars, Flame Wars, Special olympics and other ways for talks and exchange of views/opinions in the Internet. In this case I mean hot heads and "traditional packages vs. Flatpak packages". |
at the moment, portable versions, appimage and obs for native packages for popular distributions are provided for linux. adding flatpak, snap, or whateverhotnewthing to this list can only increase the workload for one active developer. |
This would come in handy on Steam Deck because you lose regularly installed packages when updating the OS there because it uses twin partition setup for updates - they both contain the same OS, on update, one of them is updated and currently active OS partition is swapped to it. There are ways to disable this behavior, but they are not recommended for general users, so having a Flatpak could be a good idea for these users. |
Immutable distros have to rely on containers. |
@soredake, why do you disagree with #309 (comment)? Rather, how? — Immuntable distributions indeed must rely upon containers: Fedora Silverblue and Kinoite's OSTree base system becomes corrupt easily if layers of standard packages are added to it. Additionally, I believe that SteamOS simply expunges them when reinitialized. |
I'm also interested in Flatpak support for Double Commander. To see how much effort this needs, I started to work on a prototype: https://github.com/TorrSamaho/org.doublecmd.Doublecmd It still has some glaring issues (crashes on exit, displays unintended mount points), but it compiles, installs and runs. |
Report about this (just run |
Thanks for the info! The |
Another update: The crash on exit does not seem to be a flatpak problem. If I build the current head of the double commander repo (8a52783) instead of tag v1.1.2, the resulting flatpak doesn't crash on exit anymore. |
Looks like application ID should be But really file manager without a full file system access is a strange thing. |
Thanks for the hint! I'll change the ID accordingly.
I'd say it depends on what you use it for. For nearly all of my use cases for Double Commander, the file access possible with flatpak is sufficient. Also, KDE's file manager has an official flatpak version: https://flathub.org/apps/org.kde.dolphin |
This comment was marked as resolved.
This comment was marked as resolved.
I think that DC as file manager should have a maximum file system access Powerful file manager is a low level system component. So there are many other problems/questions:
|
I changed the ID (https://github.com/TorrSamaho/io.github.doublecmd.DoubleCommander), replaced the file permissions with
To open files with their corresponding default tool, one could perhaps use
If you want to call an arbitrary external application,
Using So, numerous features certainly require some changes in DC to go through the corresponding portals. |
Great, thanks! This both seems to work nicely for me with your changes.
This does not work on my end yet. When I try to delete a file with DC in flatpak, I get the error: "Can not delete ... to trash! Delete directly?"
Added. With this added, I could configure my external editor by setting path to |
Where this file is located (home directory, external drive, etc)? In some places trash cannot work. |
My home direction. I tried a file in BTW: I also updated the flatpak to use 5712b87 instead of 1.1.2 and removed my patch for the mount hiding. |
(cherry picked from commit 1efe181)
(cherry picked from commit 5712b87)
I added |
Thanks! I just tested this with a file in my home directory and it works like a charm. I updated my flatpak version to use dd9704a. |
Another thought: Unsurprisingly, "Open with" doesn't work in the flatpak version yet. One could perhaps redirect this to If that works, perhaps |
(cherry picked from commit dd9704a)
Since 1.1.3 has all the flatpak changes, I updated the flatpak version to use DC 1.1.3. |
I noticed that the flatpak needs more permissions ( |
FIX: Memory leak (cherry picked from commit 6ee8e81)
Thanks for the "open with" flatpak changes, i.e. 0750dc3! Work nicely for me after some quick testing. |
I updated the flatpak to DC 1.1.4 and org.kde.Platform 5.15-23.08. |
Small issue I noticed with the DC flatpak: It does not seem to map |
Is it possible to install via Flatpak somehow? Thanks! |
No. I don't recommend to use Flatpak version. It has a limited functionality due flatpak sandbox. |
Repository version doesn’t start on my fedora 39 :( |
Open a new issue about it. Execute DC from terminal and copy output there. |
If you don't mind building the Flatpak yourself, you can install it. Assuming you have
Then you should be able to build and install the Flatpak as follows:
After this, the Flatpak version of DC should be installed as any other Flatpak. You can start like any other Flatpak, e.g., from the terminal using
Sure, the Flatpak version has limited functionality, but depending on what you use DC for, it may be completely sufficient. The main reason I started working on this is that there is no DC package for Ubuntu 22.04 ARM64 (at least there was none the last time I checked). BTW: I updated the flatpak to DC 1.1.9. |
As I see the Ubuntu repository contains packages for ARMv7 and ARM64 (GTK2/Qt5): maybe it would be more logical to ask to enable the building of packages for these architectures in the openSUSE Build Service? |
What about SteamDeck users and users of other OSes? |
@skobkin, see https://openbuildservice.org/help/manuals/obs-user-guide/cha.obs.package_formats#sec.pkgfmt.flatpak (per https://openbuildservice.org/2021/02/18/introducing-flatpak-builds/). OBS consequently appears to support Flatpak compilation too. |
I found it on Wiki: "Steam Deck runs SteamOS version 3, based on the Arch Linux operating system". I'm not ready to consider Arch Linux an exotic system.
How can I find out the full list of restrictions? (In the context that we are dealing with a file manager.) |
@Skif-off |
@Skif-off, https://docs.flatpak.org/en/latest/sandbox-permissions-reference.html#f4 contains and links to a comprehensive list of inode access restrictions. https://docs.flatpak.org/en/latest/sandbox-permissions.html#filesystem-access explains why. Device access should not be problematic, per https://docs.flatpak.org/en/latest/sandbox-permissions.html#device-access. Arch Linux isn't an exotic system, but SteamOS is in some regards, being a new immutable fork of it. |
For those interested to discuss adding DoubleCmd to Flathub repository. You're welcome to join this related new discussion at #1486 |
My rationale is available at https://github.com/safing/portmaster-packaging/issues/43#issue-956312378#:~:text=It%20shall%20allow,to%20support%20this. I am thankful for any assistance.
The text was updated successfully, but these errors were encountered: