Skip to content
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

Tentative snap building. #1210

Closed
wants to merge 4 commits into from
Closed

Tentative snap building. #1210

wants to merge 4 commits into from

Conversation

cprov
Copy link

@cprov cprov commented Aug 13, 2016

This is a proposal to include snap building support upstream.

It's temporarily pushing to the jq-cprov snap (installed with snap install jq-cprov), but once agreed upstream, we can easily setup the jq snap along with the most suitable auto-building infrastructure (Travis or Launchpad).

@coveralls
Copy link

coveralls commented Aug 13, 2016

Coverage Status

Coverage remained the same at 85.333% when pulling 76ec3df on celso-providelo:master into 0b82185 on stedolan:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 85.333% when pulling 5dfb7a0 on cprov:master into 0b82185 on stedolan:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 85.333% when pulling 4a09502 on cprov:master into 0b82185 on stedolan:master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 85.333% when pulling e67f632 on cprov:master into 0b82185 on stedolan:master.

@nicowilliams
Copy link
Contributor

Hi. What is "snap building"?

@dequis
Copy link

dequis commented Jan 24, 2017

Some fancy new packaging system that uses containers or something http://snapcraft.io/

This drops OSX building in travis, reindents travis.yml for no reason, and includes some random encrypted file which won't work if merged because travis encryption keys are tied to specific repos. If this were my project I'd say "no, thanks"

@dequis
Copy link

dequis commented Jan 24, 2017

Actually there's #1201 which looks significantly better, doesn't mess with travis or attempt to do uploads by itself, just adds build targets. And it looks like @cprov was interested in letting that other PR supersede this one.

@nicowilliams
Copy link
Contributor

OK, thanks.

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

Successfully merging this pull request may close these issues.

4 participants