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

Updated zenoh-c to latest commit #164

Merged
merged 3 commits into from
May 7, 2024
Merged

Updated zenoh-c to latest commit #164

merged 3 commits into from
May 7, 2024

Conversation

imstevenpmwork
Copy link
Contributor

Hi @Yadunund!
We're submitting this PR to ensure we use the latest zenoh-c commit before proceeding with our planned testing

Copy link
Member

@Yadunund Yadunund left a 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?

Copy link
Member

@Yadunund Yadunund left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry minor nitpick.

zenoh_c_vendor/CMakeLists.txt Outdated Show resolved Hide resolved
@clalancette
Copy link
Collaborator

I think I'm a little confused by this PR. As far as I understood, if we bump to the latest zenoh-c, then some of our tests stop working. We are still debugging that. Am I misunderstanding, or did we solve that particular issue?

@imstevenpmwork
Copy link
Contributor Author

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.

Copy link
Member

@Yadunund Yadunund left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@Yadunund Yadunund merged commit d5948d0 into ros2:yadu/bump_zenoh_with_tokio May 7, 2024
6 checks passed
Yadunund added a commit that referenced this pull request May 8, 2024
* 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>
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.

3 participants