feat: validate devel bases with craft-application
Metadata
Current evaluation
Superseded by canonical/snapcraft#4796, merged as commit 5dad5eb62, which upgraded craft-application to 2.7.0 and adopted its devel-base validation.
Resolution: closed superseded
Reason: The maintainer explicitly closed this draft in favor of #4796. That PR merged via commit 5dad5eb62 and delivered the intended validation through the craft-application 2.7.0 upgrade.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [ ] 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 `tox run -m lint`?
- [ ] Have you successfully run `tox run -e test-py310`? (supported versions: `py39`, `py310`, `py311`, `py312`)
-----
For https://github.com/canonical/craft-application/pull/302 and https://github.com/canonical/craft-application/issues/298
This has the side-effect that `build-base: devel` is no longer required for `core24` snaps.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed superseded | Superseded by canonical/snapcraft#4796, merged as commit 5dad5eb62, which upgraded craft-application to 2.7.0 and adopted its devel-base validation. | |
| stealth/space-bunny-alpha | — | closed superseded | Closed as superseded by canonical/snapcraft#4796, which merged the craft-application 2.7.0 upgrade in commit 5dad5eb62 and adopted the devel-base validation approach. | |
| qwen/qwen3.6-35b-a3b | — | — | Superseded by pull request #4796. The changes to validate devel bases with craft-application and remove the build-base: devel requirement for core24 snaps were closed in favor of the newer implementation. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Closed and superseded by PR #4796. Originally drafted to wait for a craft-application release before validating devel bases, changes were consolidated into the newer PR instead. |
Update history
No update history recorded yet.
Related work
-
Superseded By:
canonical/snapcraft#4796
(confidence 100%)
Merged replacement that upgraded craft-application and implemented the adopted devel-base validation approach.
-
Related To:
canonical/craft-application#302
(confidence 95%)
Upstream craft-application change targeted by the original validation work.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1634 feat: validate devel bases with craft-application | charmcraft | closed | Closed unmerged as unnecessary after craft-application changed; generic devel-base validation had already landed upstream in canonical/craft-application#302. | |
| #532 feat: validate devel bases with craft-application | rockcraft | closed | Closed unmerged after #569 bumped craft-application to 2.7.0, delivering the devel-base validation planned in #532. | |
| #302 feat!: validate devel bases | craft-application | merged | Merged PR #302 as aeae554e, adding Craft-wide validation that development bases use the matching devel build-base and fixing #298 (CRAFT-2675). | |
| #1366 build(deps): bump craft-application to 7.2.1 | rockcraft | closed | Closed unmerged as superseded by canonical/snapcraft#6409, which merged the same craft-application 7.2.1 bump in commit 25dd38674. | |
| #5164 feat: validate build commands before execution | snapcraft | merged | Merged #5164 (CRAFT-3293), adding pre-execution validation for app build-command definitions and closing #4999; the validation shipped in Snapcraft 8.5.1. | |
| #4653 feat(craft-application): support remote build | snapcraft | closed | Closed unmerged; superseded by canonical/snapcraft#4664 (commit 57d184af7), which landed the same craft-application remote build support for core24. | |
| #4562 Validity of `base` is not checked when using `--destructive-mode` | snapcraft | closed | Base validation in destructive mode was fixed in Snapcraft 8.1 via craft-application 2.0 and canonical/craft-application PR #263. | |
| #4560 feat(craft-application): initial support | snapcraft | merged | Merged PR #4560 as 98550856e, adding initial craft-application support with fallback to Snapcraft's existing logic. |