Skip to content

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

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

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

Conversation

@fujitatomoya
Copy link
Contributor

@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().
957e960 
Signed-off-by: Tomoya Fujita <Tomoya.Fujita@sony.com>
@fujitatomoya fujitatomoya self-assigned this Aug 20, 2025
@fujitatomoya fujitatomoya requested a review from alsora August 20, 2025 07:47
fujitatomoya
fujitatomoya commented Aug 20, 2025
View reviewed changes
Copy link
Contributor 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 along with ros2/rclcpp#2848

@fujitatomoya fujitatomoya mentioned this pull request Aug 20, 2025
Merged
@fujitatomoya
Copy link
Contributor Author

fujitatomoya commented Aug 20, 2025

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

ahcorde commented Aug 22, 2025

Pulls: #574
Gist: https://gist.githubusercontent.com/ahcorde/3e74ee3cbec2ff86bf6e8c6972af2ac1/raw/5d59e8a50ae63a7d99e317d017926ee5e5089f07/ros2.repos
BUILD args: --packages-above-and-dependencies test_communication test_rclcpp
TEST args: --packages-above test_communication test_rclcpp
ROS Distro: rolling
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/16778

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

@fujitatomoya
Copy link
Contributor Author

fujitatomoya commented Aug 23, 2025

see ros2/message_filters#201 (comment)

@fujitatomoya fujitatomoya merged commit 51679c2 into rolling Aug 23, 2025
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