Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Add the packaging metadata to build the parity snap #5496

Merged
merged 2 commits into from
Apr 25, 2017
Merged

Add the packaging metadata to build the parity snap #5496

merged 2 commits into from
Apr 25, 2017

Conversation

come-maiz
Copy link

This package will let you publish the latest parity in the Ubuntu store, and from there reach many users on all the supported Ubuntu versions, and more Linux distributions in progress.

@parity-cla-bot
Copy link

It looks like @ElOpio hasn'signed our Contributor License Agreement, yet.

The purpose of a CLA is to ensure that the guardian of a project's outputs has the necessary ownership or grants of rights over all contributions to allow them to distribute under the chosen licence.
Wikipedia

You can read and sign our full Contributor License Agreement at the following URL: https://cla.parity.io

Once you've signed, plesae reply to this thread with [clabot:check] to prove it.

Many thanks,

Parity Technologies CLA Bot

@come-maiz
Copy link
Author

come-maiz commented Apr 24, 2017

Hello!

Here at Ubuntu we are working on snaps, a new packaging format that will let you publish your software directly to the official store without having to make a complex debian package, without having to land it in debian and without having to wait for the 6 months release cycle of ubuntu. You can automate the delivery of parity in a secure way, so your users get new versions following your own timeline. And very important to your use case is that all the users will get an automatic update every time you push a new revision.

You can read more about snaps here: http://snapcraft.io/

To test this in an Ubuntu 16.04 machine:

$ sudo apt install git snapcraft
$ git clone https://github.com/elopio/parity
$ cd parity
$ git checkout snapcraft
$ cd scripts
$ snapcraft
$ sudo snap install *.snap --dangerous
$ parity

(The --dangerous flag is because the snap that you build locally is not signed by the store. Once you push it to the store, your users will just have to do $ sudo snap install parity)

If you have any questions, please let me know.

@come-maiz
Copy link
Author

[clabot:check]

@parity-cla-bot
Copy link

It looks like @ElOpio signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

@come-maiz come-maiz mentioned this pull request Apr 24, 2017
@5chdn 5chdn added A0-pleasereview 🤓 Pull request needs code review. M1-ci 🙉 Continuous integration. M2-installer 📲 Installers for MacOS and Windows. A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). labels Apr 25, 2017
Copy link
Contributor

@5chdn 5chdn left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good and works on my ubuntu server 16.4. Is it a hard requirement to put it in the snap/ subdirectory? Can we move it to scripts/ and tell snapcraft about the yaml file location?

@arkpar arkpar added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Apr 25, 2017
@arkpar arkpar merged commit 80a58ee into openethereum:master Apr 25, 2017
@arkpar
Copy link
Collaborator

arkpar commented Apr 25, 2017

Thank you for the contribution

@come-maiz
Copy link
Author

Thanks for trying it @5chdn !

It's not a hard requirement. I moved the yaml to scripts, but now you have to call snapcraft from that directory. I'll update the instructions in this PR.

We are building a service for automatic builds in build.snapcraft.io. There it's a requirement to have the yaml either in the root, or in the snap dir. But we can talk about that later, and move back the yaml if you are interested in using it.

For now, I would recommend you to push an initial snap to the edge channel in the store, so you can start experimenting: https://snapcraft.io/docs/build-snaps/publish

The snaps in the edge channel are not visible in user searches, so it's great for early adopters and quick feedback. We can make a call for testing in the ubuntu community too to get more testers.

pura vida

@come-maiz come-maiz deleted the snapcraft branch April 25, 2017 14:22
@come-maiz
Copy link
Author

Thanks for the merge @arkpar :D

@General-Beck General-Beck self-assigned this Apr 25, 2017
@come-maiz
Copy link
Author

Oh, hey @General-Beck! I didn't notice you had already pushed the parity snap to the store back then in 15.04. Many things have changed when we reached the LTS. Is there something I can do to help you with the new snap?

@General-Beck
Copy link
Contributor

@ElOpio Tomorrow I will add to CI. And if there are questions, I will certainly turn to you for advice

@arkpar arkpar mentioned this pull request Apr 28, 2017
@General-Beck
Copy link
Contributor

@come-maiz
Copy link
Author

Sorry for the late reply @General-Beck. Take a look at #5781.

5chdn referenced this pull request Jun 7, 2017
create snapcraft.yaml  for snapcraft.io
@5chdn
Copy link
Contributor

5chdn commented Jun 7, 2017

@General-Beck @devops-parity can we please use scripts/snapcraft.yaml - there is no point in having this script in multiple locations.

c5c1978 ???

@come-maiz
Copy link
Author

@5chdn the idea of moving it to snap/snapcraft.yaml is to use the build.snapcraft.io service for continuous delivery, instead of having to take care of building and publishing on your own CI system.

@General-Beck
Copy link
Contributor

General-Beck commented Jun 7, 2017

@5chdn https://build.snapcraft.io/user/paritytech/parity
@ElOpio canonical-web-and-design/build.snapcraft.io#833 build errors and while we use our CI. As there is no choice of tags or brunch on the site https://build.snapcraft.io

@5chdn
Copy link
Contributor

5chdn commented Jun 14, 2017

That's fine as long as we remove the script from scripts/ again. This repository is already confusing enough.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). A8-looksgood 🦄 Pull request is reviewed well. M1-ci 🙉 Continuous integration. M2-installer 📲 Installers for MacOS and Windows.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants