Skip to content

Support wildcards in rosdep#614

Merged
tfoote merged 1 commit intoros-infrastructure:ros2from
rkent:rosdep-wildcards
Nov 12, 2025
Merged

Support wildcards in rosdep#614
tfoote merged 1 commit intoros-infrastructure:ros2from
rkent:rosdep-wildcards

Conversation

@rkent
Copy link
Contributor

@rkent rkent commented Nov 12, 2025

Fixes #611

Generated-by: Portions of this commit may include code completion from github.copilot version 1.372.0 or later

It took me many hours to understand the rosdep parsing, but in the end the fix seems to be this simple one liner.

I've got a run available with this change at http://pr-rosindex.rosdabbler.com

Signed-off-by: R Kent James <kent@caspia.com>
Generated-by: Portions of this commit may include code completion from github.copilot version 1.372.0 or later
Copy link
Member

@tfoote tfoote left a comment

Choose a reason for hiding this comment

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

Yeah, this should be a bit improvement. It was added at somepoint but rosindex wasn't updated.

This is much better

@tfoote tfoote merged commit 79e54ab into ros-infrastructure:ros2 Nov 12, 2025
1 check passed
@rkent rkent deleted the rosdep-wildcards branch December 10, 2025 00:45
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.

Mapping of rosdep key to system packages on system dep package doesn't consider platform version wildcard

2 participants