plainbox-provider plugin: rewrite python shebangs
Metadata
Current evaluation
Merged rewrite of Python shebangs in the plainbox-provider plugin to use /usr/bin/env python. Approved by two reviewers across five files. Resolves LP: #1626587. Missing unit tests were noted but acknowledged as unavailable.
Suggested action: —
No scores available.
Issue body
LP: #1626587
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Merged rewrite of Python shebangs in the plainbox-provider plugin to use /usr/bin/env python. Approved by two reviewers across five files. Resolves LP: #1626587. Missing unit tests were noted but acknowledged as unavailable. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a rewrite of Python shebangs in the plainbox-provider plugin to use the in-snap Python. A reviewer requested unit tests, but the author noted the plugin lacks existing test coverage. The change was merged without additional tests. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged the plainbox-provider plugin to rewrite Python shebangs. Reviewer requested unit tests and regex fixes, but the author noted the plugin lacks existing tests. The PR was merged without these changes. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #830 python plugin: only replace proper shebangs. | snapcraft | merged | Merged a python plugin fix that restricts shebang replacement to file beginnings. Approved by one reviewer, the change resolves plainbox snap conflicts with a +6/-1 diff across two files. | |
| #1626587 plainbox-provider plugin python shebangs are mismatched | snapcraft (launchpad) | closed | Closed without resolution. The plainbox-provider plugin did not align Python shebangs with the python plugin, causing Snapcraft 2.17 build failures. No fix was applied, indicating the report was abandoned or superseded by later updates. | |
| #680 Rewrite shebangs generated by the python plugins. | snapcraft | merged | Merged changes that rewrite Python plugin shebangs to use env instead of hardcoded paths, resolving Launchpad bug #1597919. After adding requested integration tests, the changes were approved and merged across 13 files. | |
| #1719 many: account for python shebang args in rewrite | snapcraft | merged | Merged after reviewer approval and passing tests. Unifies Python shebang rewrite logic to properly handle arguments instead of relying on /usr/bin/env. Resolves Launchpad bugs #1730472 and #1673451. | |
| #311 plugins: fix python shebang | rockcraft | merged | Merged a fix for the Python shebang in plugins. The change replaces hardcoded interpreters with ${PARTS_PYTHON_INTERPRETER} to respect user overrides. Approved by two reviewers and merged. |