-
Notifications
You must be signed in to change notification settings - Fork 36
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
Updated zenoh-c to latest commit #164
Updated zenoh-c to latest commit #164
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you also update the comment above the with PR # associated with this commit hash?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry minor nitpick.
I think I'm a little confused by this PR. As far as I understood, if we bump to the latest |
Hello @clalancette! This pull request targets the branch to evaluate rmw with Tokio zenoh-c. Although the current hash already corresponds to Tokio zenoh-c, it's a few weeks outdated. Hence, we believe it's prudent to update this branch to the latest Tokio zenoh-c version and rerun the tests to assess any potential improvements or regressions. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM.
* Sync past tokio bump and drop thread restriction Signed-off-by: Yadunund <yadunund@openrobotics.org> * Bump zenoh-c to eclipse-zenoh/zenoh-c#307 * Updated zenoh-c to latest commit (#164) * updated zenoh-c to latest commit * updated zenoh-c commit comment * Updated comment in cmake --------- Signed-off-by: Yadunund <yadunund@openrobotics.org> Co-authored-by: Julien Enoch <julien.e@zettascale.tech> Co-authored-by: Steven Palma <imstevenpmwork@ieee.org>
Hi @Yadunund!
We're submitting this PR to ensure we use the latest zenoh-c commit before proceeding with our planned testing