Skip to content

Use NodeInterfaces for tf2_ros constructors - #96

Open
mjcarroll wants to merge 1 commit into
rollingfrom
fix-tf2-node-interfaces
Open

mjcarroll wants to merge 1 commit into
rollingfrom
fix-tf2-node-interfaces

Conversation

@mjcarroll

@mjcarroll mjcarroll commented Sep 25, 2026 •

Copy link
Copy Markdown
Member

The rolling binary build of this package has been failing since late August on all four platforms. geometry2 0.46.2 removed the deprecated tf2_ros constructors that take a node pointer or separate node interfaces (ros2/geometry2#940), and this package still uses them.

Assisted-by: Claude Opus 5

geometry2 0.46.2 removed the deprecated tf2_ros constructors that took a
node pointer or separate node interfaces (ros2/geometry2#940), which breaks
the rolling binary build for this package.

Pass the node itself instead: rclcpp::node_interfaces::NodeInterfaces has an
implicit constructor taking a node reference.

Assisted-by: Claude Opus 5 <noreply@anthropic.com>
Signed-off-by: Michael Carroll <mjcarroll.oss@gmail.com>
@mjcarroll mjcarroll self-assigned this Sep 25, 2026
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