-
Notifications
You must be signed in to change notification settings - Fork 163
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
Build on each git push and upload an AppImage #338
base: master
Are you sure you want to change the base?
Conversation
The built AppImage identifies its version number as
Why is it "dirty"? |
I tested the lftp AppImage on Debian Jessie, Arch Linux, and Devuan Jessie, and it works perfectly in all of them. Lftp is awesome...and AppImage is pure magic. I'm also curious as to why it labels itself as "dirty" given that all the functionality I've tested is working. |
"dirty" means that some of the sources were modified, probably by gnulib-tool. |
It would be logical to use just compiled lftp instead of wget and curl. Besides it would be a final QA test. |
Is sudo really needed? |
Do you see what we could do differently in the |
You can try to checkout after bootstrap.
вс, 23 апр. 2017, 14:47 probonopd <[email protected]>:
… "dirty" means that some of the sources were modified, probably by
gnulib-tool.
Do you see what I could do differently in the .travis.yml to avoid being
marked "dirty"?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#338 (comment)>, or mute
the thread
<https://github.com/notifications/unsubscribe-auth/AA67XCJj7QsSjZYUgIXtnJoOIjCdk0IWks5ryzpEgaJpZM4NFLpi>
.
|
Yes, if you want to run AppImages, since for strange reasons Travis CI denies access for the |
Can you give me the commands to replace
please? |
Just use lftpget instead of wget. Or
For upload, lftp would be not so useful (yet), as it does not currently save the PUT reply body anywhere. |
In dc95b47 I have created contrib/Makefile to install the icon and desktop file. |
I'm integrating appimage build rules into Makefile now. Can you suggest a way to build appimage with modular lftp (configure --with-modules)? Probably it should have LFTP_MODULE_PATH set to the /tmp/.mount_*/usr/lib/lftp/$(VERSION) somehow. |
This PR, when merged, will compile this application on Travis CI upon each
git push
, and upload an AppImage to a temporary download URL on transfer.sh (available for 14 days). The download URL is toward the end of each Travis CI build log of each build (see below for how to set up automatic uploading to your GitHub Releases page).For this to work, you need to enable Travis CI for your repository as described here prior to merging this, if you haven't already done so.
Providing an AppImage would have, among others, these advantages:
appimaged
Here is an overview of projects that are already distributing upstream-provided, official AppImages.
Please note: Instead of storing AppImage builds temporarily for 14 days each on transfer.sh, you could use GitHub Releases to store the binaries permanently. This way, they would be visible on the Releases page of your project. This is what I recommend. See https://docs.travis-ci.com/user/deployment/releases/. If you want to do this for continuous builds, also see https://github.com/probonopd/uploadtool.
If you have questions, AppImage developers are on #AppImage on irc.freenode.net.