Skip to content

get rid of deprecated rclcpp::spin_some().#734

Merged
fujitatomoya merged 1 commit intorollingfrom
fujitatomoya/deprecate-global-spin-some-all
Aug 23, 2025
Merged

get rid of deprecated rclcpp::spin_some().#734
fujitatomoya merged 1 commit intorollingfrom
fujitatomoya/deprecate-global-spin-some-all

Conversation

@fujitatomoya
Copy link
Collaborator

@fujitatomoya fujitatomoya commented Aug 20, 2025

Description

This depends on ros2/rclcpp#2848

Fixes # (issue)

Is this user-facing behavior change?

No

Did you use Generative AI?

No

Additional Information

@fujitatomoya
get rid of deprecated rclcpp::spin_some().
5f60934 
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
fujitatomoya
fujitatomoya commented Aug 20, 2025
View reviewed changes
Copy link
Collaborator Author

@fujitatomoya fujitatomoya left a comment

Choose a reason for hiding this comment

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

@alsora requesting review

This was referenced Aug 20, 2025
Merged
Merged
@fujitatomoya
Copy link
Collaborator Author

fujitatomoya commented Aug 20, 2025

Pulls: ros2/rclcpp#2848, ros2/message_filters#201, ros2/system_tests#574, #734, ros2/examples#422, ros2/geometry2#821
Gist: https://gist.githubusercontent.com/fujitatomoya/729c2c9e3f31efc10217a07d09b682c5/raw/80936104a2cae37018c8996fb8071ceffb79df0c/ros2.repos
BUILD args: --packages-above-and-dependencies rclcpp rclcpp_action rclcpp_lifecycle message_filters test_communication test_rclcpp pendulum_control dummy_map_server dummy_sensors examples_rclcpp_minimal_publisher examples_rclcpp_minimal_action_client tf2_ros
TEST args: --packages-above rclcpp rclcpp_action rclcpp_lifecycle message_filters test_communication test_rclcpp pendulum_control dummy_map_server dummy_sensors examples_rclcpp_minimal_publisher examples_rclcpp_minimal_action_client tf2_ros
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/16754

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

This was referenced Aug 20, 2025
Merged
Merged
Merged
@ahcorde
Copy link
Contributor

ahcorde commented Aug 22, 2025

Pulls: #734
Gist: https://gist.githubusercontent.com/ahcorde/efd852127eb08a7837e109208e120cf0/raw/4ba02ec668874e183fb8ac6ae7dc8f70b01d4aee/ros2.repos
BUILD args: --packages-above-and-dependencies dummy_map_server dummy_sensors pendulum_control
TEST args: --packages-above dummy_map_server dummy_sensors pendulum_control
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/16773

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

@fujitatomoya
Copy link
Collaborator Author

fujitatomoya commented Aug 23, 2025

see ros2/message_filters#201 (comment)

@fujitatomoya fujitatomoya merged commit 472a479 into rolling Aug 23, 2025
3 checks passed
@ahcorde ahcorde deleted the fujitatomoya/deprecate-global-spin-some-all branch August 25, 2025 07:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ahcorde ahcorde ahcorde approved these changes

@alsora alsora alsora approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@fujitatomoya @ahcorde @alsora