chore(legacy): remove script entry point
Metadata
Current evaluation
Merged as f8cf41382, removing the obsolete legacy script entry point to resolve confusion raised in #5778.
Resolution: closed resolved
Reason: PR #5821 was approved by two reviewers and merged; commit f8cf41382 deleted the legacy entry point from pyproject.toml. The change followed the #5778 finding that the compatibility path was dead code because the distributed snap did not expose it.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [ ] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)?
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [ ] Have you successfully run `make lint`?
- [ ] Have you successfully run `make test`?
---
This removes an old legacy entry point that caused some confusion (see #5778).
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged as f8cf41382, removing the obsolete legacy script entry point to resolve confusion raised in #5778. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5821 as f8cf41382, deleting the obsolete legacy script entry point that caused confusion in #5778. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as f8cf41382: removed the obsolete legacy script entry point that caused confusion in #5778, deleting one line after approval by two reviewers. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged removal of a legacy script entry point causing user confusion. Approved by two reviewers, passed CI checks, and deleted a single line in one file to clean up the codebase. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged maintainer bepri's cleanup to remove a legacy script entry point that caused confusion, addressing issue #5778. The change was approved and merged without additional comments. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/snapcraft#5778
(confidence 98%)
Closed unmerged after determining the distributed snap did not expose the legacy Python entry point, motivating this removal.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1765 chore: remove classic entrypoint | charmcraft | merged | Merged charmcraft#1765 as commit ffca2b8e, removing the obsolete classic entrypoint left over from the craft-application migration. | |
| #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. | |
| #5731 chore(legacy-cli): remove unused help.py and version.py, drop related test | snapcraft | closed | Closed unmerged as superseded by merged #5777, which combined this cleanup with #5773 and removed the unused legacy help/version commands, crediting both contributors. | |
| #5103 chore(merge): 7.5.7 into main | snapcraft | merged | Merged 7.5.7 into main in fa644b44c as a no-op because equivalent fixes already existed on hotfix/8.4. | |
| #5540 chore: resolve deprecation warnings | snapcraft | merged | Merged PR #5540 as commit 93e9c9237, fixing deprecation warnings and closing #5539. | |
| #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. | |
| #5056 chore(merge): 8.4.1 into main | snapcraft | merged | Merged release branch 8.4.1 into main via PR #5056 in commit e4026155d. | |
| #5361 chore(merge): 8.7.4 into main | snapcraft | merged | Merged 8.7.2, 8.7.3, and 8.7.4 into main ahead of the 8.8 cut via approved PR #5361 (commit 2d63f0e8a, CRAFT-4291). | |
| #5133 chore(merge): 8.4.4 into main | snapcraft | closed | Merged 8.4.4 maintenance changes into main via #5134, commit 187187d53. | |
| #1040 chore(merge): 2.4.3 into main | craft-parts | merged | Merged version 2.4.3 into main via PR #1040, landing as commit fab7fa92 after two reviewer approvals. |