← Back to issue list

Fix rosdep ROS_PYTHON_VERSION

View original Github issue

Metadata

Project
snapcraft
Number
#3346
Type
pull request
State
merged
Author
artivis
Labels
Created
Updated
Closed

Current evaluation

Merged. Sets ROS_PYTHON_VERSION to 3 for ROS 2 rosdep usage, correcting underlying behavior rather than merely silencing warnings. Approved by reviewers and merged following low-risk assessment.

Suggested action:

No scores available.

Issue body

Sets `ROS_PYTHON_VERSION=3` if `Rosdep` used for ROS 2 (colcon plugin V1). Signed-off-by: artivis <deray.jeremie@gmail.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. Sets ROS_PYTHON_VERSION to 3 for ROS 2 rosdep usage, correcting underlying behavior rather than merely silencing warnings. Approved by reviewers and merged following low-risk assessment.
qwen3.6-35b-a3b-mtp-q6 Merged fix for rosdep ROS_PYTHON_VERSION in the ROS 2 colcon plugin V1. Sets the variable to 3 to resolve warnings and correct bad behavior, superseding a previous incomplete fix. Approved as low risk.
qwen3.6-35b-a3b-mtp-q6 Merged. Sets ROS_PYTHON_VERSION to 3 for ROS 2 colcon plugin V1, fixing rosdep configuration. Supersedes PR #3324 by resolving a warning and correcting existing behavior. Approved by maintainers.

Update history

No update history recorded yet.

Related issues

Issue Project State Summary Similarity
#3324 set ROS_PYTHON_VERSION for rosdep snapcraft merged Merged a two-line update setting ROS_PYTHON_VERSION=2 for the rosdep environment, resolving a harmless defaulting warning. Approved by two reviewers and passed CI checks.
89%