chore: remove snapcraft_legacy
Metadata
Current evaluation
Merged #6241 via commits ffb164b7a and 995f0a3bc, deleting snapcraft_legacy, obsolete specs, and legacy-only dependencies after its entry points were removed.
Resolution: closed resolved
Reason: PR #6241 was approved and merged, removing the legacy implementation and its dependencies in commits ffb164b7a and 995f0a3bc. The author confirmed the remaining spread failures were unrelated.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Removes snapcraft_legacy, now that the code has no entry points.
Also removes the old specs folder and drops snapcraft_legacy dependencies.
---
- [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 |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged #6241 via commits ffb164b7a and 995f0a3bc, deleting snapcraft_legacy, obsolete specs, and legacy-only dependencies after its entry points were removed. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged #6241 removes snapcraft_legacy, obsolete specs, and associated dependencies after confirming no entry points remained. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR removed obsolete snapcraft_legacy code, legacy specs, and dependencies; commit ffb164b7a. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged removal of the snapcraft_legacy module, old specs folder, and dependencies after verifying no remaining entry points. Approved by two reviewers; noted unrelated integration test failures. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged by maintainer mr-cal after removing snapcraft_legacy code, the old specs folder, and associated dependencies following the elimination of entry points. CI spread failures were deemed unrelated. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #6035 chore!: remove deprecated commands | snapcraft | merged | Merged as 3623601d7, removing deprecated Snapcraft commands and options and resolving #5674 (SNAPCRAFT-1196). | |
| #5994 chore!: remove legacy remote builder | snapcraft | merged | Merged PR #5994 as b08f1a66e, removing the legacy remote builder and closing #5676 for Snapcraft 9.0. | |
| #1078 chore!: remove deprecated APIs | craft-application | merged | Merged PR #1078 as 3334f13e, removing eight deprecated APIs and fixing #1063. All CI checks passed, with local Snapcraft verification. | |
| #6472 chore: remove unused set_legacy_env | snapcraft | closed | Closed unmerged; author superseded it by reworking branch layout to work/refactor/* and pushing directly to canonical/snapcraft. | |
| #5777 chore(legacy-cli): remove unused help and version commands | snapcraft | closed | Merged #5777, removing unused legacy CLI help/version commands after confirming snapcraft_legacy was not exposed in official releases; it combined #5731 and #5773. | |
| #6470 chore: remove unused errors | snapcraft | closed | PR removing unused SnapcraftError subclasses was closed unmerged; superseded by reworking the branch layout and pushing directly to canonical/snapcraft. | |
| #6469 chore: remove unused _read_config | snapcraft | closed | Closed without merging; superseded by a reworked branch pushed directly to canonical/snapcraft under work/refactor/*. | |
| #6019 chore!: drop core20 | snapcraft | merged | Merged PR #6019 as fa7952b4d, removing core20 build support and moving dropped entry points to snapcraft_legacy; generic store commands remained for later migration. | |
| #6074 chore!: remove default command | snapcraft | merged | PR #6074 merged as db7c53f21, removing bare snapcraft packing for #5673; #6178 later reverted the change in e5aaf0683. | |
| #5183 chore(merge): 8.5.1 into main | snapcraft | merged | Merged Snapcraft 8.5.1 into main via #5183 (merge commit bfb63fa79), after approval by two reviewers. |