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

Conversation

andy5995
Copy link

fixes #77

@@ -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

@andy5995 andy5995 closed this Nov 14, 2022
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.

Missing link to AppRun file in manual packaging doc
3 participants