← Back to issue list

Remove legacy help and version code

View original Github issue

Metadata

Project
snapcraft
Number
#5682
Type
issue
State
closed
Author
mr-cal
Labels
good first issue Status: Triaged Type: Cleanup
Created
Updated
Closed

Current evaluation

Merged PR #5777 removed the unused snapcraft_legacy help/version commands and their tests; the legacy package was later removed wholesale for Snapcraft 9.

Resolution: closed resolved

Reason: PR #5777 directly implemented and merged the requested cleanup. Commits ffb164b7a and 995f0a3bc later removed snapcraft_legacy and its dependencies entirely.

Evaluated by: mr-cal-bot

No scores available.

Issue body

### What needs to get done Remove the `help` and `version` commands in `snapcraft_legacy` and all accompanying unit tests. We can keep any existing spread tests, since these commands should work even when in a directory with a core20 snapcraft.yaml file. ### Why it needs to get done The `help` and `version` commands in `snapcraft_legacy` are no longer used.

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Merged PR #5777 removed the unused snapcraft_legacy help/version commands and their tests; the legacy package was later removed wholesale for Snapcraft 9.
stealth/space-bunny-alpha — closed resolved Merged PR #5777 removed snapcraft_legacy's unused help/version commands and unit tests, resolving #5682 ahead of the planned Snapcraft 9 removal.
stealth/space-bunny-alpha — closed resolved Merged PR #5777 "chore(legacy-cli): remove unused help and version commands" deleted snapcraft_legacy's unused help.py/version.py and their unit tests, combining #5731 and #5773; issue closed by maintainer mr-cal ahead of Snapcraft 9.
stealth/space-bunny-alpha — closed resolved Closed as resolved: merged PR #5777 removed the unused legacy help/version commands and their tests, ahead of the planned removal of snapcraft_legacy in Snapcraft 9.
qwen/qwen3.6-35b-a3b — — Closed as obsolete. The entire snapcraft_legacy module is being removed in Snapcraft 9, rendering the specific cleanup of legacy help and version commands unnecessary. No pull request was merged.
qwen/qwen3.6-35b-a3b — — Closed and abandoned as the entire snapcraft_legacy module will be removed in Snapcraft 9, rendering this cleanup obsolete. No pull request was merged.
qwen/qwen3.6-35b-a3b — — Legacy help and version code cleanup was abandoned. The maintainer closed the issue, noting snapcraft_legacy will be entirely removed in Snapcraft 9, rendering the changes unnecessary.
qwen3.6-35b-a3b-mtp-q6 — — Closed as obsolete. Maintainers plan to completely remove snapcraft_legacy in Snapcraft 9, rendering this cleanup task unnecessary. No code changes were implemented.

Update history

No update history recorded yet.

Related work

Related issues

Issue Project State Summary Similarity
#5773 Remove help and version commands from snapcraft_legacy snapcraft closed Approved PR #5773 was closed unmerged and superseded by merged PR #5777, which combined its cleanup with #5731 and removed the unused legacy help/version commands.
86%
#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.
81%
#3735 commands: help for all legacy commands snapcraft merged Merged PR #3735, adding help passthrough for all legacy Snapcraft commands; merged as commit 8f8b2df42.
74%
#1793261 Vendor legacy snapcraft to maintain backward compatibility snapcraft (launchpad) closed Legacy Snapcraft was vendored into the snapcraft snap, dispatching projects without bases to it. Delivered via merged PRs canonical/snapcraft#2283 (commit f9592939a, override-build refactor) and #2285 (commit 5c1ee3b88).
73%
#5982 Trim down snapcraft_legacy and plan migration effort snapcraft closed Superseded by a revised plan to migrate store commands first, then delete snapcraft_legacy; the package was later removed in ffb164b7a and 995f0a3bc.
73%
#3609 tests: move legacy snapcraft tests (CRAFT-728) snapcraft merged Merged PR #3609 in commit a71aeff54, moving snapcraft_legacy unit tests from tests/unit to tests/legacy to match the package rename.
72%
#6241 chore: remove snapcraft_legacy snapcraft merged Merged #6241 via commits ffb164b7a and 995f0a3bc, deleting snapcraft_legacy, obsolete specs, and legacy-only dependencies after its entry points were removed.
72%
#3608 many: rename snapcraft package to snapcraft_legacy (CRAFT-726) snapcraft merged Merged PR #3608 renamed the existing snapcraft package to snapcraft_legacy, freeing the snapcraft name for a craft-library-based application while leaving its entry point unchanged.
71%
#2285 snapcraft snap: vendor legacy snapcraft snapcraft merged Merged in canonical/snapcraft#2285 (commit 5c1ee3b88), vendoring legacy Snapcraft and dispatching no-base projects to it to preserve backward compatibility.
70%
#4194 Migrate snapcraft_legacy subcommands to use craft-cli snapcraft closed Requested legacy-handler bypass was abandoned; verbosity support landed in #5646/#5680, and snapcraft_legacy was later removed in ffb164b7a.
70%