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

Add snap packaging support #21

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

casualscripter
Copy link

Solution for #20 .

With these changes, you can build a snap package by yourself.

For an official snap package there is left a little more work.

Snapcraft expects single quotes instead of double quotes.
This file was just generated automatically with a fixed version of tools/snap/generate_dnsplugins_snapcraft.sh from the Certbot project.
Actually snap-constraints.txt is just a copy of requirements.txt.
Should problems occur generating with tools/pinning/current/repin.sh form the Certbot project could be a better solution!?
Addes some instructions for building, installing and using your own snap package.
Added instructions for uninstalling old cerbot-packages, installing snap and certbot from snap.
@nils-wisiol
Copy link
Contributor

Thanks, will have a closer look at this after the holiday time!

@nils-wisiol
Copy link
Contributor

Thanks for this PR, I would like to merge this. However, given that I'm not familiar with snap, can some sort of testing be integrated into this? Ideally there would be a Github Action that guarantees that everything works as expected.

Also, can we bump this to Ubuntu 22.04 and Certbot 2? (See also #22, #23)

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.

2 participants