Skip to content

Conversation

@fujitatomoya
Copy link

Description

follow @clalancette 's fix on ros2/rosidl_dynamic_typesupport#17

Fixes # (issue)

Is this user-facing behavior change?

Not really for the user.

Did you use Generative AI?

Additional Information

@fujitatomoya
Copy link
Author

@clalancette does this also need to be applied?

@fujitatomoya fujitatomoya self-assigned this Oct 29, 2025
@clalancette
Copy link
Contributor

@clalancette does this also need to be applied?

Yes. It is a bit confusing, but this one is rosidl_dynamic_fastrtps_typesupport_fastrtps, while ros2/rosidl_dynamic_typesupport#17 is for rosidl_dynamic_typesupport (different packages).

@clalancette
Copy link
Contributor

Oh wait, sorry. I got confused. I see you opened this one, but this is a duplicate of #9 . So I'm going to close this one out in favor of that.

@fujitatomoya
Copy link
Author

@clalancette oh you already have 👍 my bad i did not see that.

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.

3 participants