Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Pass &interrupt_guard_condition_ to the executor_notifier_ #26

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

3473f
Copy link

@3473f 3473f commented Jul 18, 2024

I realize this repo is depricated, however as the executor is not avaialable for humble and the main branch not building for humble anymore, I thought I would open this pull request as I got it running through an easy bug-fix.

The get() method was causing the code not to compile.
@mauropasse
Copy link
Collaborator

As a comment, there's a branch humble-future of this repo which should have the fix for humble.
Ideally we have one branch per ROS2 distro (because main can't satisfy the APIs compatibility of all distributions).

@3473f
Copy link
Author

3473f commented Jul 18, 2024

Thanks for the pointer! I thought that main was targeting humble as mentioned here, while the humble-future branch requires the changes to rclcpp and rclc mentioned in Readme. I followed the example here with the osrf/ros:humble container and wasn't able to compile it without this fix.

Should I close this PR? or maybe leave it as a pointer to anyone trying to compile it for humble in the future.

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