docker: don't rely on snapcraft-classic
Metadata
Current evaluation
Merged a fix updating Dockerfiles to replace a broken snapcraft-classic symlink with a script that correctly invokes Python and Snapcraft from the snap. Approved by one reviewer and passed CI.
Suggested action: —
No scores available.
Issue body
- [x] Have you followed the [guidelines for contributing](https://github.com/snapcore/snapcraft/blob/master/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] If this is a bugfix. Have you checked that there is a bug report open for the issue you are trying to fix on [bug reports](https://bugs.launchpad.net/snapcraft)?
- [x] If this is a new feature. Have you discussed the design on the [forum](https://forum.snapcraft.io)?
- [x] Have you successfully run `./runtests.sh static`?
- [x] Have you successfully run `./runtests.sh unit`?
-----
The Dockerfiles currently create a symlink from `/snap/bin/snapcraft` to `/snap/snapcraft/current/bin/snapcraft-classic`. However, `snapcraft-classic` no longer exists. We can't just symlink to `snapcraft` either, or Python isn't found.
Instead of creating a symlink, write a simple script to run the Python (and Snapcraft) out of the snap.
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged a fix updating Dockerfiles to replace a broken snapcraft-classic symlink with a script that correctly invokes Python and Snapcraft from the snap. Approved by one reviewer and passed CI. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a Dockerfile fix replacing a broken symlink with a wrapper script to execute Python and Snapcraft directly, resolving compatibility issues after snapcraft-classic was deprecated. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a Dockerfile fix replacing a broken symlink to the removed snapcraft-classic with a simple script. The script correctly invokes Python and Snapcraft from the snap, resolving build failures. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #3210 docker: install snapd dependency | snapcraft | merged | Merged. Adds snapd installation to Docker images to satisfy snapcraft's new snap pack requirement. Approved by two reviewers, passed CI, and modified five files. | |
| #2245 docker: support for testing snapcraft in proposed | snapcraft | merged | Merged additions for a xenial-proposed snapcraft test Dockerfile and cleanup of the standard Dockerfile. Changes received approval, passed CI checks, and were validated through successful container execution. |