support assumes field in snapcraft.yaml
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Canonical Click Reviewers tools (obsolete) |
Fix Released
|
Wishlist
|
Jamie Strandboge | ||
Snapcraft |
Fix Released
|
Wishlist
|
Sergio Schvezov | ||
snapcraft (Ubuntu) |
Fix Released
|
Undecided
|
Unassigned | ||
Xenial |
Fix Released
|
Undecided
|
Unassigned | ||
Yakkety |
Fix Released
|
Undecided
|
Unassigned |
Bug Description
[Impact]
* snapd now supports checking for snaps before installing that they don't require features that the current snapd/core doesn't support,
* snapd compares a feature tag list in snap.yaml under the entry name "assumes" to the its own internal list of supported features,
* for this to work snapcraft needs to support copying a list of strings entry called "assumes" from snapcraft.yaml to the snap.yaml of the built snap
* The nodejs plugin is hard-coded to use version 4.2.2. This should be configurable via a `node-engine` config parameter.
[Test Case]
1. Run `snapcraft` on a snapcraft.yaml with a set of keywords in `assumes`.
2. Make sure they make it to snap.yaml
[Regression Potential]
* Very low. This is a new feature.
description: | updated |
Changed in snapcraft: | |
milestone: | none → 2.10 |
description: | updated |
Changed in snapcraft: | |
status: | New → In Progress |
assignee: | nobody → Sergio Schvezov (sergiusens) |
Changed in snapcraft: | |
status: | In Progress → Fix Committed |
Changed in snapcraft (Ubuntu Xenial): | |
milestone: | none → xenial-updates |
Changed in snapcraft: | |
status: | Fix Committed → Fix Released |
Changed in click-reviewers-tools: | |
status: | New → Fix Committed |
importance: | Undecided → Wishlist |
assignee: | nobody → Jamie Strandboge (jdstrand) |
Changed in click-reviewers-tools: | |
status: | Fix Committed → Fix Released |
Thanks Samuele! For completeness, would you mind adding a YAML snippet to the description to serve as an example?