Ros plugin: handle stderr separated from stdout, fixes #5600
Metadata
Current evaluation
Merged PR #5615 as commit 721d86958, separating rosdep stderr from parsed stdout and fixing ROS Jazzy dependency resolution in Snapcraft 8.10.2.
Resolution: closed resolved
Reason: PR #5615 was merged as commit 721d86958, preventing rosdep stderr warnings from being parsed as dependency output. This resolved issue #5600 and was included in Snapcraft 8.10.2 via #5620.
Evaluated by: mr-cal-bot
No scores available.
Issue body
- [ ] Have you followed the [guidelines for contributing](https://github.com/canonical/snapcraft/blob/main/CONTRIBUTING.md)?
- [x] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
- [x] Have you successfully run `make lint`?
- [ ] Have you successfully run `make test`?
---
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5615 as commit 721d86958, separating rosdep stderr from parsed stdout and fixing ROS Jazzy dependency resolution in Snapcraft 8.10.2. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #5615 as commit 721d86958, separating rosdep stderr warnings from parsed stdout; it fixes #5600 and was released in Snapcraft 8.10.2. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged in #5615 (721d86958), retargeting the ROS stderr/stdout fix from #5609 to the 8.10 hotfix branch and fixing #5600. | |
| stealth/space-bunny-alpha | — | closed resolved | Merged as 721d86958, separating stderr from stdout in the ROS plugin to fix rosdep failures on Jazzy/core24 (#5600), ahead of planned Snapcraft 8.10.2. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged after approval and CI validation. The change fixes #5600 by correctly separating stderr from stdout in the Ros plugin. Maintainer landed the fix to address a Jazzy/core24 issue ahead of the 8.10.2 release. | |
| qwen3.6-35b-a3b-mtp-q6 | — | — | Merged to fix Ros plugin stderr handling when separated from stdout, resolving #5600. Following review and version targeting discussion, the change was landed for the 8.10.2 release. |
Update history
No update history recorded yet.
Related work
-
Related To:
canonical/snapcraft#5600
(confidence 100%)
ROS Jazzy rosdep failure fixed by separating stderr from stdout.
-
Related To:
canonical/snapcraft#5609
(confidence 98%)
Earlier PR for the same fix; #5615 landed after addressing its review feedback.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #5609 Ros plugin: handle stderr separated from stdout, fixes #5600 | snapcraft | closed | ROS stderr/stdout handling was retargeted to #5615, merged as 721d86958, and released in Snapcraft 8.10.2; #5609 was then closed. | |
| #5600 ROS2 Jazzy plugins fails with rosdep | snapcraft | closed | ROS2 Jazzy builds failed because rosdep's pkg_resources DeprecationWarning on stderr tripped the ROS plugin's output parsing. Hotfix PR #5609 was retargeted to #5615, merged as 721d86958, and shipped in Snapcraft 8.10.2. |