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

Document --appimage-extract-and-run #65

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Conversation

probonopd
Copy link
Member

@probonopd probonopd commented Aug 10, 2024

Closes #64

Only merge after discussion.

If I remember correctly, this flag is not documented by design since we don't want users to use it if they just have a broken FUSE setup. Using this command line option results in a degraded user experience with longer application start times, wasteful temporary space usage, etc.

Normal users should just fix their FUSE setup instead.

If I also remember correctly, this was mainly introduced for Docker setups, because Docker has issues with FUSE (at least without special workarounds). But Docker users prefer the use of environment variables.

Alternatives:

  • Keep it undocumented
  • Remove it and only offer an environment variable

In any case, it should be noted that this is only a feature of this runtime, and other AppImage runtimes may or may not have similar options. So users MUST NOT assume that every AppImage supports this command line option.

Copy link

Build for testing:
artifacts
Use at your own risk.

Copy link

Build for testing:
artifacts
Use at your own risk.

@TheAssassin
Copy link
Member

I'm open to removing it, but we should then document the environment variable somewhere (and make it a mandatory feature of the spec perhaps). I don't mind having the parameter either, though, since after all it's not a lot of extra code.

@Samueru-sama
Copy link

I don't like the idea of removing the option, that will break a lot of workflows that use it.

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.

--appimage-extract-and-run is undocumented
3 participants