← Back to issue list

snap: prepare override scripts to allow rebuilding

View original Github issue

Metadata

Project
snapcraft
Number
#2223
Type
pull request
State
merged
Author
sergiusens
Labels
Created
Updated
Closed

Current evaluation

Merged into master. Updated snapcraft.yaml override scripts to fix ctypes linking and PyYAML patching, enabling easier project rebuilding. Approved by reviewer and passed CI checks.

Suggested action:

No scores available.

Issue body

This projects snapcraft.yaml does not allow for easy rebuilding given the use of linking and patching to make ctypes work properly and the required patches to PyYAML. Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com> - [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)? - [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? - [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)? - [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)? - [x] Have you successfully run `./runtests.sh static`? - [x] Have you successfully run `./runtests.sh tests/unit`? -----

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b Merged into master. Updated snapcraft.yaml override scripts to fix ctypes linking and PyYAML patching, enabling easier project rebuilding. Approved by reviewer and passed CI checks.
qwen3.6-35b-a3b-mtp-q6 Merged updates to snapcraft.yaml override scripts, enabling easier rebuilding by resolving ctypes linking and PyYAML patching. References bug #1789206. Codecov noted a minor coverage decrease upon integration.
qwen3.6-35b-a3b-mtp-q6 Merged into master. Updated snapcraft.yaml override scripts to fix ctypes linking and PyYAML patching, enabling straightforward project rebuilding. All checks passed.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#2283 snapcraft snap: refactor override-build into a script snapcraft merged Merged refactoring that extracts override-build into a standalone script to reduce snapcraft.yaml duplication and support internal vendoring. Approved by reviewer, passed CI, and modified two files.
78%
#2425 snap: re-add pyc files for snapcraft snapcraft merged Merged to master after approval and passing CI. The change re-adds pyc files for snapcraft to resolve Launchpad bug #1808199, modifying two files with minor adjustments.
72%
#1601 snapcraft.yaml: don't re-use build dir snapcraft merged Merged a fix for a source-build directory clash in snapcraft.yaml. The build directory was renamed to apt-build to avoid scriptlet conflicts. Approved by reviewers and passed CI, resolving the failure.
71%
#1915 snap: patch ctypes for the snap snapcraft merged Merged a maintainer pull request patching ctypes for the snap. The change updated 13 files, adding 71 and removing 97 lines. Approved by one reviewer and passed CI checks before integration.
70%
#1073 project: snapcraft.yaml in a snap directory snapcraft merged Merged into master. Defaults new projects to place snapcraft.yaml inside a snap directory while retaining support for legacy locations. Approved and passed CI checks.
70%