vadrian89 / cumulus-qt

Qt port for Cumulus
GNU General Public License v3.0
40 stars 10 forks source link

Generate and upload AppImage for each git push #10

Closed probonopd closed 7 years ago

probonopd commented 7 years ago

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:

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 would like to see only one entry for the Pull Request in your project's history, then please enable this GitHub functionality on your repo. It allows you to squash (combine) the commits when merging.

If you have questions, AppImage developers are on #AppImage on irc.freenode.net.

probonopd commented 7 years ago

Remaining issue: application wants to write to Cumulus-local.db in the same directory as the application itself, which will not work, since that location is read-only. Should write somewhere in $HOME instead. This probably needs to be changed in the application's source code?

vadrian89 commented 7 years ago

Thanks for your contribution, I will merge after I update the location of the db file.

Pe 29 iun. 2017 12:17 a.m., "probonopd" notifications@github.com a scris:

Remaining issue: application wants to write to Cumulus-local.db in the same directory as the application itself, which will not work, since that location is read-only. Should write somewhere in $HOME instead.

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/vadrian89/cumulus-qt/pull/10#issuecomment-311794697, or mute the thread https://github.com/notifications/unsubscribe-auth/AHzIhr1GVYMUjmvmyYq8YEoc0XrS78fFks5sIsL1gaJpZM4OIi41 .

vadrian89 commented 7 years ago

I need to delay a bit on this pull request.
Besides the database file, I also need to change how the application manages the "Start on login" option...aswell some testing.....so by the end of the week I hope I will merge it.