catkin plugin: do not overwrite the PYTHONPATH
Metadata
Current evaluation
Merged PR #324 fixed the catkin plugin to preserve setup.sh’s PYTHONPATH instead of overwriting it, resolving LP #1544790.
Resolution: closed resolved
Reason: PR #324 was merged in commit 0adb5cc16 with implementation commit c845aefdd. The fix retains setup.sh’s catkin Python path and resolved the failing ROS test.
Evaluated by: mr-cal-bot
No scores available.
Issue body
LP: #1544790
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #324 fixed the catkin plugin to preserve setup.sh’s PYTHONPATH instead of overwriting it, resolving LP #1544790. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #324 fixed the catkin plugin to preserve setup.sh's PYTHONPATH, resolving LP #1544790. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #324 fixed the catkin plugin to preserve setup.sh’s PYTHONPATH instead of overwriting it, resolving ROS test failure LP: #1544790. | |
| qwen/qwen3.6-35b-a3b | — | — | 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. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged a fix for LP #1544790 that stops the catkin plugin from overwriting PYTHONPATH. Preserving the variable set by setup.sh resolves the ROS test failure and related bug. |
Update history
No update history recorded yet.
Related work
-
Related To:
Launchpad#1544790
(confidence 100%)
Original bug report linked from the merged PR.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #1522 catkin plugin: only append PYTHONPATH if set | snapcraft | merged | Merged in commit a2440c986: the Catkin plugin now appends PYTHONPATH only when set, preventing empty path segments; environment-output coverage was added. | |
| #1714333 Catkin plugin: empty segment added to PYTHONPATH | snapcraft (launchpad) | closed | Fixed by snapcraft PR #1522, merged as commit a2440c986 ("catkin plugin: only append PYTHONPATH if set"), which makes the Catkin plugin append the existing PYTHONPATH only when it is set, avoiding an empty path segment. | |
| #1354 catkin plugin: fix pythonpath for catkin_find | snapcraft | closed | Closed unmerged after inactivity; its linked catkin-underlay failure was later fixed by merged PR #1526 in commit ae9fd598f. | |
| #1239 catkin plugin: create completely valid environment | snapcraft | merged | Merged PR #1239 (commit 449ad2634) fixing LP: #1674487: catkin plugin now sets LD_LIBRARY_PATH and a valid ROS_HOME before sourcing setup.sh, so build-time env matches runtime. |