Skip to content

Fix wrong dependency#3133

Merged
skyegalaxy merged 1 commit intoros2:rollingfrom
mini-1235:fix/dependency
Apr 18, 2026
Merged

Fix wrong dependency#3133
skyegalaxy merged 1 commit intoros2:rollingfrom
mini-1235:fix/dependency

Conversation

@mini-1235
Copy link
Copy Markdown
Contributor

Description

Related to #3124

Is this user-facing behavior change?

Did you use Generative AI?

Additional Information

Signed-off-by: Maurice <mauricepurnawan@gmail.com>
Copy link
Copy Markdown
Member

@skyegalaxy skyegalaxy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch! was able to repro this locally and your patch fixed it

@skyegalaxy
Copy link
Copy Markdown
Member

Pulls: #3133
Gist: https://gist.githubusercontent.com/skyegalaxy/a06e24ffc311e4bbf3ddfd3b83a2abaf/raw/7f0a02de69404f690149127f729ba520cbc96b9e/fix_dependency_repos
BUILD args: --packages-above-and-dependencies rclcpp
TEST args: --packages-above rclcpp
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/18995/

  • Linux Build Status
  • Linux-aarch64 Build Status
  • Linux-rhel Build Status
  • Windows Build Status

@skyegalaxy skyegalaxy merged commit 6572239 into ros2:rolling Apr 18, 2026
3 of 4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants