docker: add core18 snap that snapcraft now uses as a base
Metadata
Current evaluation
Merged PR #2883 as commit 6bd9dac10, adding core18 to Docker images as Snapcraft's base and updating related comments.
Resolution: closed resolved
Reason: PR #2883 was merged into Snapcraft as commit 6bd9dac10. The change adds the core18 snap used by Snapcraft's Docker base and corrects related comments.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Also update comments for correctness and syntax.
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 |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #2883 as commit 6bd9dac10, adding core18 to Docker images as Snapcraft's base and updating related comments. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as 6bd9dac10; added the core18 snap used by Snapcraft's Docker base and corrected related comments. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in #2883 as 6bd9dac10, adding the core18 snap used by Snapcraft’s Docker base and correcting related comments. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as 6bd9dac10 in #2883, adding the core18 snap used as Snapcraft's Docker base and updating related comments for accuracy and syntax. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged the addition of the core18 snap as snapcraft's Docker base, updating related comments for accuracy. Approved by one reviewer, the change was integrated after a minor authorship clarification. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged updates adding core18 snap as the Docker base and correcting related comments. Changes were cherry-picked from #2851 and approved by maintainers. |
Update history
No update history recorded yet.
Related work
-
Related To:
snapcraft#2851
(confidence 90%)
A reviewer referenced #2851 while discussing the merge and authorship of this follow-up.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2712 snap: migrate to core18 | snapcraft | merged | Merged PR #2712 by maintainer sergiusens moved Snapcraft's own snap from base core to core18, landing as commit bd5be51a3 (8 files, +95/-42); it superseded the earlier closed attempt PR #2444. | |
| #1814017 Add docker image for core18 building | snapcraft (launchpad) | open | Request for a core18-capable snapcraft Docker image for CI; in-repo Dockerfiles gained core18 support (b982555e1) then were removed in favor of Canonical's snapcraft-rocks images, and core18 now needs EOL 7.x snapcraft. | |
| #2444 snap: move to core18 | snapcraft | closed | Superseded by merged PR #2712, which moved Snapcraft's snap from core to core18 in commit bd5be51a3. | |
| #1846195 provide snapcraft docker image for core18 | snapcraft (launchpad) | open | Request (2019) for an official snapcraft Docker image able to build base: core18 snaps; unmaintained since 2020, in-repo Dockerfiles were removed in 24e70d6db in favor of canonical/snapcraft-rocks, and core18 now needs EOL Snapcraft 7.5.x. | |
| #3579 snap: move base to core20 (CRAFT-509) | snapcraft | merged | Merged PR #3579 via ed2baf8bd, moving Snapcraft's own snap to the core20 base; implementation commit 18a974e01. | |
| #4177 core18: removal | snapcraft | merged | Merged PR snapcore/snapcraft#4177 (May 2023) dropped core18 base support and its unit/spread tests, fixing #4165 via commits 40136ea9b, 77edc734b and ed423b6a8; core18 was EOL. | |
| #2720 remote build: switch from core to core18 | snapcraft | merged | Merged PR #2720 switched remote builds from core to core18, aligning them with Snapcraft edge; landed as commit d2b36b2f4. | |
| #3210 docker: install snapd dependency | snapcraft | merged | Merged PR #3210 in commit 42d3861a, adding snapd to Snapcraft Docker images so required snap pack operations continue working. | |
| #1939713 snapcraft init create base: core18 snap | snapcraft (launchpad) | closed | Snapcraft init was updated to scaffold core20 instead of core18 by merged canonical/snapcraft#3587 (merge commit f91632aab). | |
| #4290 docker: add core22 base | snapcraft | closed | Rejected as incompatible with legacy behavior; the Dockerfile was later removed and snapcraft-rocks documented as the official replacement in #5063 (24e70d6db). |