schema: build-base support for the snapd type
Metadata
Current evaluation
Merged into master. Implements build-base keyword support for the snapd snap type, enabling its construction. Approved by reviewer, passed CI, and updated four files.
Suggested action: —
No scores available.
Issue body
Enabled building these two snap types using the build-base keyword.
Signed-off-by: Sergio Schvezov <sergio.schvezov@canonical.com>
- [ ] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `./runtests.sh static`?
- [ ] Have you successfully run `./runtests.sh tests/unit`?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged into master. Implements build-base keyword support for the snapd snap type, enabling its construction. Approved by reviewer, passed CI, and updated four files. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master, adding build-base keyword support for the snapd snap type in the schema. This enables building snapd snaps with the specified build-base configuration. Codecov verified the merge with negligible coverage impact. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged into master, adding build-base keyword support for the snapd snap type in the schema. The change increased code coverage by less than 0.01%. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2700 schema: build-base support for the kernel type | snapcraft | merged | Merged into master. Adds build-base schema support for kernel type snaps, noting they do not require a base until hooks are implemented. Included in the next release, with a reviewer noting a related dependency PR is also needed. | |
| #4580 fix: do not propagate build-base to snap.yaml | snapcraft | merged | Merged a fix preventing build-base from propagating to snap.yaml. The pull request received two approvals, passed CI checks and code coverage, and was successfully integrated. | |
| #3729 parts: handle build base | snapcraft | merged | Merged into snapcraft/7.0. Implements build-base handling for parts, configuring the correct build environment by using the declared build-base or falling back to the package name for base-type projects. Approved and passed CI. | |
| #4868 Allow specifying build-base in kernel snap | snapcraft | closed | Implemented as a prototype in a feature branch via pull request #4914, enabling kernel snaps to specify build-bases using the standard cross-craft naming system. | |
| #4983 Install base snap if declared as `build-base` | snapcraft | closed | Closed as superseded. Base and build-base combination restrictions in #4878 render this enhancement moot. The author acknowledged the change and implemented a workaround. Implementation may be revisited for kernel snap interim base support. | |
| #2309 snap: improve early base detection logic | snapcraft | merged | Merged. Resolved early base detection failure in build providers by updating logic to account for snapcraft's non-project directory execution. Approved by reviewer, passed CI, and merged with minimal changes. | |
| #2174 build_providers: inject snaps when running from a snap | snapcraft | merged | Merged. Updates build providers to inject snapcraft and its base from the host when running inside the snapcraft snap, ensuring parity with non-snap environments. Approved by reviewer and passed CI checks. | |
| #3014 meta: migrate get_build_base to Snap | snapcraft | merged | Merged after approval and passing CI. Migrated get_build_base from Projectinfo to Snap, simplifying test fixtures and removing plugin dependencies on self.project.info across 61 files. | |
| #2344 schema: remove the deprecated snap keyword for bases | snapcraft | merged | Merged into master, the schema no longer uses the deprecated snap keyword for bases, replacing it with prime. Approved by two reviewers, passed CI, and increased coverage by 0.04%. | |
| #3836 snapcraft/meta: make Manifest.base optional | snapcraft | merged | Merged a fix making Manifest.base optional to resolve a validation error crashing manifest generation for core22 base snaps. Approved and merged after passing CI. Reviewers suggested adding a spread test for base building cases. |