meta: always generate snapcraft-runner to workaround classic PATH bug
Metadata
Current evaluation
Merged a workaround that always generates snapcraft-runner to resolve classic snap PATH bugs. Unit and integration tests were added. This temporary fix addresses snapd issue LP#1860369 and will be removed once snapd resolves the underlying problem.
Suggested action: —
No scores available.
Issue body
Snapcraft previously generated wrappers for most commands. Once
snapcraft became more conserative about generating unnecessary wrappers,
issues began to surface with regard to PATH being set incorrectly
for classic snap apps that did not use shell.
To work around the issue, this commit adds an empty snapcraft-runner to
the command-chain for all apps, where they typically would have none.
Once this issue is resolved in snapd, we can probably remove this
workaround.
LP#: 1860369
Signed-off-by: Chris Patterson <chris.patterson@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 |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged a workaround that always generates snapcraft-runner to resolve classic snap PATH bugs. Unit and integration tests were added. This temporary fix addresses snapd issue LP#1860369 and will be removed once snapd resolves the underlying problem. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Adds an empty snapcraft-runner to all app command-chains to workaround a classic snap PATH bug in snapd (LP#1860369). Tests were added per reviewer feedback. A separate broken compatibility check was noted but deferred to a future PR. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged. Adds an empty snapcraft-runner to all apps to work around a classic snap PATH bug in snapd (LP#1860369). Unit and integration tests were added per reviewer request. The workaround remains until snapd resolves the issue. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #2891 meta: always generate snapcraft-runner to workaround classic (#2889) | snapcraft | merged | Merged as #2889. Introduces an empty snapcraft-runner to the command-chain to fix incorrect PATH resolution for classic snap apps. Includes unit tests and configurable fixtures. Approved by reviewers and passed CI. |