-
Notifications
You must be signed in to change notification settings - Fork 58
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
snapcraft.yaml: update to work with modern snapcraft #132
Conversation
spread is currently not build with snapcraft 5.0 (the default). This commit fixes this by moving to `base: core20`. Note that this also requires moving to `go.mod`.
@@ -12,6 +12,7 @@ description: | | |||
|
|||
confinement: strict | |||
grade: stable | |||
base: core20 | |||
|
|||
apps: | |||
spread: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
adding an entry for personal-files to be able to easily use the google backend would be nice.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent suggestion, I think this would ideally be a followup?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I addressed this in PR #161 now.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would update the version to something relevant (not blocking on it) and consider another PR to move to core22. This works for now.
@@ -1,5 +1,5 @@ | |||
name: spread | |||
version: 2018.08.21 | |||
version: 2020.08.19 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
can we use git
as version here?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I will do a followup with this change and also the move to core22. This PR got (soft) approval by Gustavo as is so I don't want to add more potentially controversial things just now :)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I addressed the version in #161 - please let me know if that matches what you had in mind and I'm happy to tweak as needed.
spread is currently not build with snapcraft 5.0 (the default).
This commit fixes this by moving to
base: core20
.Note that this also requires moving to
go.mod
hand hence it includes #129.