Add the packaging metadata to build the synapse snap #1158

Closed
wants to merge 9 commits into
from

Conversation

Projects
None yet
3 participants

elopio commented Oct 5, 2016

This is a package for the secure installation of apps that works in most Linux distributions.
Landing it upstream will enable builds for adventurous users.

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

elopio commented Oct 5, 2016

You can find more information about snapcraft here: http://snapcraft.io/

If you upload the snap to the ubuntu store you will get a great way to deliver the software, and get transactional automatic updates. I've been testing the snap [1]. It still needs more testing, and some help from you to understand the remaining errors; but it seems to me good enough to publish in the edge channel, which will make it really easy to do crowd-testing before the stable release.

If you have any doubts or comments, please let me know.

[1] https://gist.github.com/elopio/dffcda326f5b51feedeb09f012ac9a44

Owner

erikjohnston commented Oct 11, 2016

Thanks for the idea! In general I've been reluctant to merge in all the various package manager metadata files. This is for two reasons:

  1. It ties the version of the metadata to the version of synapse
  2. There are so many of them

I'd be happier to either put this under contrib/ as more an example, or have the snap file in a separate repo with the appropriate instructions added to the README.

elopio commented Oct 11, 2016

Thanks for reviewing. I'm reaching out to a few upstreams to gather feedback that will help us sort our backlog, so I appreciate a lot your comments.

In our sprint next week we have scheduled a discussion about the version string in the yaml, because it's a pain point also for me. We are probably going to add a --version flag to the snapcraft command, so it can be overwritten, but we'll have to discuss some other options first. For now, we can use a template like in https://github.com/syncthing/syncthing/pull/3636/files. I can also move it to the contrib folder if you prefer. There are some other projects like https://github.com/nextcloud/nextcloud-snap using a separate repo, which is a nice solution too. But I can't create repos in your organization, somebody from your team would have to do that.

As a side note, I've been experimenting on the other matrix projects. Do you think it would be useful if I send snapcraft metadata to them too? I'm personally interested in matrix because I want to steer my hackerspace away from telegram, as it never released the server source code. I think having snaps for riot and a few server implementations will make it easier. It's not a requirement of course, and we'll start testing it soon anyway, but I like snaps a lot :D

elopio commented Nov 1, 2016

@erikjohnston ping. I'm wondering what to do next here.

Owner

erikjohnston commented Dec 5, 2016

Sorry for dropping this on the floor, its been quite hectic here. I've created: https://github.com/matrix-org/package-synapse-snap so feel free to PR against that and I'd be happy to merge that there.

Owner

erikjohnston commented Dec 5, 2016

(I shall close this for now, but please do PR against the repo! I would but I have no experience with snap)

elopio commented Dec 5, 2016

@erikjohnston will do. Thanks.

@elopio elopio deleted the elopio:snapcraft branch Dec 5, 2016

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