Add informations/refactor update page #82
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
When I discovered the update process of the AppImages, I was confused about a few things
I modified the documentation to include information about these things, in the hope that new users will understand more clearly the update process and how to implement it right
Here is the list of commits with my justifications for making them :
a0c63b4 : My workflow initially used
mv
to rename the appimage after its creation, which caused the update to fail. I figured it out by looking at the binary data of the .zsync file, to see that the original filename was included in it, and that I needed to rename the appimage from the packaging tool directlye52efab : I didn't personally have this problem, but I can see people confusing appimagetool/linuxdeploy/etc... with these "external tools" you talk about (which are AppImageUpdate/etc...). I don't know if I phrased it the best way possible, but it seems more understandable from my PoV
09662d1 : appimage-builder not being in the list might confuse people (it did for me at first) and they may think that appimage-builder is yet another tool to do yet another job. Since you already talk about other projects such as electron-builder, I thought appimage-builder would also fit in
c4df8dd : Addition to a0c63b4 to tell packagers how they should rename the AppImage if they want to
d1356a2 : TheAssassin explained me this on #appimage. I thought it it had its place in the documentation
043c99d : the travis links seems to be 404 now, so I changed them to link to the github
37725cc : All this information (that was here before + what I added) do not seem to be related to updates with external tools, so I moved it to its own paragraph before that
Feel free to disagree with any change I made, I'm open to discussion !
(I know that, taken one by one, these things aren't serious problems, but together they may cause some headache understanding the update process, and well, this project is about making packaging easy, right ?)