Skip to content

tf2_ros: Fix deprecated subscriber callbacks#448

Merged
ahcorde merged 2 commits intoros2from
aprotyas/deprecate-shared-ptr-sub-callback
Aug 23, 2021
Merged

tf2_ros: Fix deprecated subscriber callbacks#448
ahcorde merged 2 commits intoros2from
aprotyas/deprecate-shared-ptr-sub-callback

Conversation

@aprotyas
Copy link
Copy Markdown
Member

ros2/rclcpp#1713 deprecates the void shared_ptr<T> subscription callback signatures, so this PR migrates away from said signatures.

Signed-off-by: Abrar Rahman Protyasha abrar@openrobotics.org

Abrar Rahman Protyasha added 2 commits August 19, 2021 21:17
Signed-off-by: Abrar Rahman Protyasha <abrar@openrobotics.org>
Signed-off-by: Abrar Rahman Protyasha <abrar@openrobotics.org>
@ahcorde
Copy link
Copy Markdown
Contributor

ahcorde commented Aug 20, 2021

--packages-above-and-dependencies tf2_ros

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@aprotyas
Copy link
Copy Markdown
Member Author

aprotyas commented Aug 20, 2021

Ignore the botched CI job. The warnings came from interactive_markers, where I didn't push the patch to deal with the deprecation and in robot_state_publisher, which used the repo from the ros2 org and not my fork.

I am running a full CI instead to target all the changes necessitated by the deprecation in ros2/rclcpp#1713.

  • Linux Build Status
  • Linux-aarch64 Build Status
  • macOS Build Status
  • Windows Build Status

@aprotyas
Copy link
Copy Markdown
Member Author

Yellow CI for Windows is unrelated to this package - originates from system_tests packages.

@ahcorde ahcorde merged commit 8b12844 into ros2 Aug 23, 2021
@delete-merged-branch delete-merged-branch bot deleted the aprotyas/deprecate-shared-ptr-sub-callback branch August 23, 2021 07:35
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