Add snapcraft.yaml #455

Merged
merged 1 commit into from May 4, 2017

Conversation

Projects
None yet
3 participants
Contributor

ChrisMacNaughton commented Apr 20, 2017

Snapcraft makes Linux packaging very simple in a cross-distro
way. This adds the snapcraft.yaml file to setup a snap of ripgrep

Add snapcraft.yaml
[Snapcraft](https://snapcraft.io/) makes Linux packaging very simple in a cross-distro
way. This adds the snapcraft.yaml file to setup a snap of ripgrep
Contributor

ChrisMacNaughton commented Apr 20, 2017

I have already registered the snap package name ripgrep, will pass ownership of that over and am happy to help with setting up automated builds!

Contributor

Phrohdoh commented May 3, 2017

Why snapcraft over flatpack?

I know next to nothing about both but there are multiple offerings available and we should probably weight the pros and cons before selecting one (I believe providing both defeats the purpose).

Owner

BurntSushi commented May 3, 2017

This isn't the place to debate snapcraft vs flatpak.

While I appreciate this PR, I know nothing about snapcraft/flatpak and I don't have any plans to use them in the immediate future, which makes it hard for me to maintain. Is it necessary for snapcraft.yml to be in this repo?

(I did make an exception for Brew since so many people are using it, although even that will eventually go away once SIMD works on stable Rust.)

Contributor

ChrisMacNaughton commented May 4, 2017

I'd rather not discuss one versus the other as well, although a benefit to snaps is that they are available by default on any Ubuntu >= 16.04.

Regarding inclusion in the repository, it is possible to use snaps without including them in the repository but it is a bit easier to deploy into a testing channel with snaps from CI if it's in tree.

@BurntSushi BurntSushi merged commit 20f7d9b into BurntSushi:master May 4, 2017

2 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
Owner

BurntSushi commented May 4, 2017

All right. I'll merge this for now and we can see how it goes. Thanks!

Contributor

ChrisMacNaughton commented May 4, 2017

@BurntSushi Feel free to ping me (or the snapcraft team) if things are needed to help maintaining it :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment