catkin plugin: fix pythonpath for catkin_find
Metadata
Current evaluation
Closed and abandoned. The author clarified the fix only resolved a highly specific local setup issue and was no longer relevant. The underlying problem persisted, prompting further troubleshooting on the forum instead of merging the changes.
Suggested action: —
No scores available.
Issue body
ROS has separate python package location that needs to be on PYTHONPATH before calling catkin_find in the generated bash script. Fixes false script failures, which the plugin interprets as not finding the specified package.
LP: [#1696014](https://bugs.launchpad.net/snapcraft/+bug/1696014)
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| qwen/qwen3.6-35b-a3b | — | — | Closed and abandoned. The author clarified the fix only resolved a highly specific local setup issue and was no longer relevant. The underlying problem persisted, prompting further troubleshooting on the forum instead of merging the changes. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Closed due to inactivity. The author clarified the fix was environment-specific and no longer relevant, though the underlying catkin underlay conflict persists and was moved to a forum discussion. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Closed without merging due to inactivity and lack of clarity. The author initially withdrew the change as environment-specific but later noted the underlying catkin PYTHONPATH issue persists. The fix was abandoned and remains unresolved. |
Update history
No update history recorded yet.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1696014 catkin plugin _Catkin.find always fails | snapcraft (launchpad) | closed | Closed. Abandoned without official resolution. A contributor identified a missing PYTHONPATH entry causing _Catkin.find to fail and proposed a patch to include the correct dist-packages directory. | |
| #230 Fix catkin plugin python calls | snapcraft | closed | Closed and abandoned by author to prevent accidental merge. Awaiting further review and a complete fix for the catkin plugin Python interpreter path and a related executable lookup issue. | |
| #324 catkin plugin: do not overwrite the PYTHONPATH | snapcraft | merged | Merged a fix for the catkin plugin that preserves the PYTHONPATH instead of overwriting it. This resolves a ROS test failure and LP: #1544790 by retaining paths set by setup.sh. Approved and merged by maintainers. | |
| #1522 catkin plugin: only append PYTHONPATH if set | snapcraft | merged | Merged fix for the Catkin plugin that only appends PYTHONPATH when set, preventing empty segments from resolving to the current directory. Added an environment test to verify the fix. Approved and merged. | |
| #1730248 catkin part error when built after python part | snapcraft (launchpad) | closed | Closed without resolution. The catkin plugin failed when built after a Python part due to incorrect path validation in is_setup() and generate_sitecustomize(), which checked the stage and install directories respectively. | |
| #1714333 Catkin plugin: empty segment added to PYTHONPATH | snapcraft (launchpad) | closed | The Catkin plugin incorrectly added an empty PYTHONPATH segment when unset, breaking snap confinement. Closed without comments or a tracked fix, likely abandoned or resolved in a later release. |