Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Use Travis CI to test flatpak recipes #1557

Merged
merged 6 commits into from
Apr 11, 2018
Merged

Use Travis CI to test flatpak recipes #1557

merged 6 commits into from
Apr 11, 2018

Conversation

amtlib-dot-dll
Copy link
Contributor

@antenore I made several changes including reverting the recipe and modifying .travis.yml for automatic flatpak builds. Is there any problem?

cc @larchunix

5b0450e#commitcomment-28518966

The recipe in this repo is pointed to the `next` branch; The recipe in https://github.com/flathub/org.remmina.Remmina/blob/master/org.remmina.Remmina.json is pointed to the latest tagged release
@antenore
Copy link
Member

Cool! Thanks!
One thought, recently we have removed the snap build from travis, because it was taking too long and often going in timeout.
So let's see how it'll work out. For snap builds we have moved the process on snapcraft. If it'll take too long is it possible to do the same on flathub?

@amtlib-dot-dll
Copy link
Contributor Author

We can disable it by removing it from the matrix section in .travis.yml file, however Travis CI allocates 50 minutes to every job and according to the log every job just takes less than 20 minutes. I haven't used snap, but I guess that snap would be even more quicker? Since flatpak requires you to compile everything by yourself.

@antenore
Copy link
Member

OK, I'm going to merge this, let's see how things will work.

@antenore antenore merged commit 6aabffc into next Apr 11, 2018
@antenore antenore removed the review label Apr 11, 2018
@amtlib-dot-dll amtlib-dot-dll deleted the patch-flatpak-travis branch April 11, 2018 08:08
@amtlib-dot-dll
Copy link
Contributor Author

Still, I wonder why the snap builds are slow 🤔

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants