Create and upload AppImage of travis build #194
Conversation
|
The pull request has been built on Travis CI successfully and has generated this AppImage, as can be seen in the build log toward the bottom: To test it: Works for me on Ubuntu 14.04 (trusty); will try on other distributions, too. |
| GIT_REV=$(git rev-parse --short HEAD) | ||
| echo $GIT_REV | ||
| sudo make install DESTDIR=/$APP/$APP.AppDir | ||
| sudo chown -R $USER /$APP |
There was a problem hiding this comment.
Why is sudo needed here? Can’t we just install to a user-writable directory (e.g. under $HOME)?
There was a problem hiding this comment.
Probably we could, but I haven't tested the functions in https://github.com/probonopd/AppImages/raw/master/functions.sh under that scenario. Feel free to start from my proposed script and change as you see fit.
There was a problem hiding this comment.
I don’t have any experience with AppImage, and I also prefer to not use sudo before this is merged.
There was a problem hiding this comment.
What is the reason you think we should avoid running sudo on travis? We are running sudo on travis already anyway...
There was a problem hiding this comment.
Because one day we would want to move to the faster container-based Travis setup where we can’t use sudo (this is currently blocked because using Trusty in Travis requires sudo, but it will be changed in the future).
|
Please squash the commits together and add a detailed commit message (the comment of this pull request should be enough). |
Bundle the Linux build artifacts on Travis CI as an AppImage and upload it to transfer.sh. Check the last lines of each Travis CI build log for a transfer.sh URL from which the AppImage can be downloaded. To run the AppImage, download it, chmod a+x ./theAppImage and run. Should work on most not-too-outdated 64-bit Linux distributions. For more information, see https://github.com/probonopd/AppImageKit/blob/master/README.md
|
Done |
|
By the way, there is a new GitHub feature which lets you squash and merge with the press of a button: |
|
Haven't had a dedicated moment to test this. I'd love to get it working for OSX as well. |
| ######################################################################## | ||
| # Package the binaries built on Travis-CI as an AppImage | ||
| # By Simon Peter 2016 | ||
| # For more information, see http://appimage.org/ | ||
| ######################################################################## | ||
|
|
||
| sudo apt-get -y install coreutils binutils python-tk |
There was a problem hiding this comment.
This should be moved to the .travis.yml files with other apt-get calls, since this will be done without sudo in the future.
|
@probonopd |
|
Then we must go back to using sudo apt-get... |
|
I don’t see the relation, besides build is passing now already. |
|
Thanks for your contribution! |
Bundle the Linux build artifacts on Travis CI as an AppImage and upload it to transfer.sh.
Check the last lines of each Travis CI build log for a transfer.sh URL from which the AppImage can be downloaded.
To run the AppImage, download it,
chmod a+x ./theAppImageand run. Should work on most not-too-outdated 64-bit Linux distributions (approximately 2014 and later).If it doesn't, then either the script needs to be fine-tuned (e.g., bundle additional or less files) and/or an older build system should be used on Travis CI (so that the resulting binaries become compatible with older target systems).
For more information, see https://github.com/probonopd/AppImageKit/blob/master/README.md