Download classic runtime by default #33
Conversation
The static runtime is still considered experimental and not nearly as battle-tested as the "classic" variant from AppImageKit. For users who wish to download the static variant instead, an environment variable was added which changes the new default behavior.
TheAssassin
force-pushed
the
download-classic-runtime
branch
from
d6e34f8
to
8fb5c02
Compare
There was a problem hiding this comment.
Please make the old runtime opt-in. After all, who does nothing still gets the old runtime from the AppImageKit repo. People who change to this repo do so because they want the static runtime.
Let's change it the other way around - static by default, old by opt-in.
This would be in line with AppImage/AppImageKit#877 (comment) - which got lots of upvotes.
The old runtime is not maintained anymore, not recommended anymore.
|
Nope. I explained the rationale very well. The vast majority of AppImages is built with regular appimagetool. This tool can't just switch to some experiment without prior notice. It was you who wanted to have no unnecessary changes before a 1.0 release. |
Show me. This repository has not been advertised anywhere on purpose. At least not by me. Which is in line with what we discussed recently. |
|
appimagetool is free to use the old appimagetool or opt out of the static runtime. Although I thought we wanted to get rid of the old dynamically linked runtime asap. |
See the upvotes on AppImage/AppImageKit#877 (comment) - people can't wait to get the static runtime due to Ubuntu not shipping libfuse2 anymore. |
It is of course still maintained actively, being part of the core AppImage repositories. Just because there are no changes doesn't mean that it is not maintained. Please stop spreading misleading information.
are not a representative statistic measure in anyway. Just looking at the download counts proves you wrong. |
TheAssassin
dismissed
probonopd’s stale review
Sudden new requirement not in line with previous discussion.
|
This PR requires discussion. |
|
@TheAssassin have you found a way to update AIL so that this PR is no longer needed? |
|
@TheAssassin can we close this for good? Thanks! |
|
Yes, let's close it. I hope to make a prerelease of AIL this week. |
The static runtime is still considered experimental and not nearly as battle-tested as the "classic" variant from AppImageKit.
For users who wish to download the static variant instead, an environment variable was added which changes the new default behavior.
This commit allows me to use this new appimagetool in linuxdeploy-plugin-appimage, which would serve as a field test for the tool itself. I do not want to make people use the static variant until it is fully compatible with everything and the specification changes have been made.