Skip to content

EventsExecutor: Handle async callbacks for services and subscriptions (backport #1478)#1613

Merged
ahcorde merged 1 commit intoros2:jazzyfrom
bjsowa:backport/jazzy/fix-event-executor-async
Mar 17, 2026
Merged

EventsExecutor: Handle async callbacks for services and subscriptions (backport #1478)#1613
ahcorde merged 1 commit intoros2:jazzyfrom
bjsowa:backport/jazzy/fix-event-executor-async

Conversation

@bjsowa
Copy link
Copy Markdown
Contributor

@bjsowa bjsowa commented Feb 22, 2026

This was never backported to Jazzy for some reason.

@bjsowa bjsowa force-pushed the backport/jazzy/fix-event-executor-async branch from 346d9c6 to d677416 Compare February 22, 2026 23:16
… (backport ros2#1478)

Co-authored-by: Alejandro Hernandez Cordero <ahcorde@gmail.com>
Co-authored-by: Błażej Sowa <bsowa123@gmail.com>
Signed-off-by: Błażej Sowa <bsowa123@gmail.com>
@bjsowa
Copy link
Copy Markdown
Contributor Author

bjsowa commented Mar 17, 2026

@ahcorde @fujitatomoya Can I get a review and/or CI run?

@ahcorde
Copy link
Copy Markdown
Contributor

ahcorde commented Mar 17, 2026

Pulls: #1613
Gist: https://gist.githubusercontent.com/ahcorde/8745ca77becaffe2076e862c120a9e70/raw/3f8f44b0ed024c27b1f42662cda701b012b1f24d/ros2.repos
BUILD args: --packages-up-to rclpy
TEST args: --packages-select rclpy
ROS Distro: jazzy
Job: ci_launcher
ci_launcher ran: https://ci.ros2.org/job/ci_launcher/18520

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

@ahcorde ahcorde merged commit 791d736 into ros2:jazzy Mar 17, 2026
3 checks passed
@bjsowa
Copy link
Copy Markdown
Contributor Author

bjsowa commented Mar 24, 2026

@ahcorde Would appreciate it if it could get released before the next sync to main.

@ahcorde
Copy link
Copy Markdown
Contributor

ahcorde commented Mar 24, 2026

@bjsowa Sure, ros/rosdistro#50357

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