← Back to issue list

build(snapcraft.yaml): update cargo

View original Github issue

Metadata

Project
snapcraft
Number
#6309
Type
pull request
State
merged
Author
mr-cal
Labels
Created
Updated
Closed

Current evaluation

Merged update to snapcraft.yaml staging cargo and consolidating build-packages, fixing a cryptography build failure introduced in #6307. Approved by two reviewers and merged after passing core CI checks.

Suggested action:

No scores available.

Issue body

Fixes a problem introduced in #6307 where snapcraft wasn't able to build cryptography. This fix is to update and stage cargo. Also bundles the build-packages into the same part for readability. Ripped from [charmcraft](https://github.com/canonical/charmcraft/blob/8a42aacf9d796e79ed987575d1d2120db14f8365/snap/snapcraft.yaml#L72-L110). --- - [x] I've followed the [contribution guidelines](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md). - [x] I've signed the [CLA](http://www.ubuntu.com/legal/contributors/). - [x] I've successfully run `make lint && make test`. - [ ] I've added or updated any relevant documentation. - [ ] In documents I changed, I [added a meta description](https://canonical-starflow.readthedocs-hosted.com/how-to/add-a-page-meta-description/) if one was missing. - [ ] I've updated the relevant release notes.

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged update to snapcraft.yaml staging cargo and consolidating build-packages, fixing a cryptography build failure introduced in #6307. Approved by two reviewers and merged after passing core CI checks.
qwen3.6-35b-a3b-mtp-q6 Merged updates to snapcraft.yaml that stage cargo to fix a cryptography build failure introduced in #6307. The change consolidates build-packages for readability and adapts the configuration from charmcraft.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#1091 snap: ensure cryptography builds correctly on all architectures charmcraft merged Merged a temporary workaround fixing cryptography build failures across all architectures for the snap. Approved by one reviewer, the change modifies two files (+48/-9 lines). A more robust fix is planned.
72%
#49 packages: add snap package support craft-parts merged Merged after implementing snap download and installation for stage and build snaps. Approved by two reviewers, passed CI, and added 1,072 lines across five files. Follows snapcraft conventions with future refactoring planned.
71%
#1326 build: bump rustc/cargo to 1.91 rockcraft merged Merged changes bumping rustc and cargo to version 1.91, adopting the installation method used by Snapcraft and Charmcraft. Approved by two reviewers and passed CI checks.
71%
#774 build(snap): update stage-packages for armhf rockcraft merged Merged an update to snap stage-packages for armhf, replacing unavailable libgpgme11 and libbtrfs0 with t64 variants. Approved by two reviewers, passed CI, and modified 2 lines in 1 file.
70%