Skip to content

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

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

get rid of deprecated rclcpp::spin_some().#422
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().
48fa406 
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

@fujitatomoya fujitatomoya requested a review from alsora August 20, 2025 07:55
@fujitatomoya fujitatomoya self-assigned this Aug 20, 2025
This was referenced Aug 20, 2025
Merged
Merged
Merged
@fujitatomoya
Copy link
Collaborator Author

fujitatomoya commented Aug 20, 2025

Pulls: ros2/rclcpp#2848, ros2/message_filters#201, ros2/system_tests#574, ros2/demos#734, #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
@ahcorde
Copy link
Contributor

ahcorde commented Aug 22, 2025

Pulls: #422
Gist: https://gist.githubusercontent.com/ahcorde/54991a1c67c2972b5fa890833d9c49e3/raw/e28fce5baaf6d4a53c5c4a170c443e393e2079be/ros2.repos
BUILD args: --packages-above-and-dependencies examples_rclcpp_minimal_publisher examples_rclcpp_minimal_action_client
TEST args: --packages-above examples_rclcpp_minimal_publisher examples_rclcpp_minimal_action_client
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/16775

  • 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 c7e7b6f into rolling Aug 23, 2025
2 of 3 checks passed
@ahcorde ahcorde deleted the fujitatomoya/deprecate-global-spin-some-all branch August 25, 2025 07:54
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

@fujitatomoya fujitatomoya

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@fujitatomoya @ahcorde @alsora