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

source/packaging-guide/manual.rst:add anchor to "this repository" #85

Closed
wants to merge 1 commit into from
Closed
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 1 addition & 1 deletion source/packaging-guide/manual.rst
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,7 @@ Create an AppDir structure that looks (as a minimum) like this::
MyApp.AppDir/usr/bin/myapp
MyApp.AppDir/usr/lib/libfoo.so.0

The :code:`AppRun` file can be a script or executable. It sets up required environment variables such as :code:`$PATH` and launches the payload application. You can write your own, but in most cases it is easiest (and most error-proof) to use a precompiled one from this repository.
The :code:`AppRun` file can be a script or executable. It sets up required environment variables such as :code:`$PATH` and launches the payload application. You can write your own, but in most cases it is easiest (and most error-proof) to use a precompiled one from `this repository <https://github.com/AppImage/AppImageBuild>`__.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That would have been AppImageKit (or, rather, its release page) if anything. However, this section is deprecated. AppImageKit's AppRun often does more harm than good. There should be some words of warning. Something like "... this one might work, but please note its limitations: [list]`.

We should open an issue about the latter point.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmmm... well I have no idea what the limitations are.

Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@TheAssassin I ran across this issue today as well. Followed the documentation, which advised me to use the AppRun binary, which I then needed to search for.

But if that's not actually what should be used, is there some recommended launch script somewhere?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is not the right place for end user support. Feel free to join our chat: https://docs.appimage.org/contact.html


Of course you can leave out the library if your app does not need one, or if all libraries your app needs are already contained in every base operating system you are targeting.

Expand Down