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

ZenohUTransport implementaton #54

Closed

Conversation

sashacmc
Copy link

Closes #51

gregmedd added a commit to gregmedd/up-transport-zenoh-cpp that referenced this pull request Jun 27, 2024
Checking changes in eclipse-uprotocol#64 enable eclipse-uprotocol#54 to proceed

Merge remote-tracking branch 'sasha/zenoh-utransport-impl' into check/add-zenohcpp-conan
@gregmedd
Copy link
Contributor

@sashacmc - #64 has been merged and confirmed working with zenoh-cpp. Two steps will be needed to update this PR to pull in those changes:

  1. Rebase onto the latest main (should be fb8581d). For merge conflicts with CMakeLists.txt, just take what is on main.
  2. Apply this patch: build_fixes.patch.tar.gz

The patch makes a few small changes that will enable the code on this branch to build.

@gregmedd
Copy link
Contributor

@sashacmc - we have diverged from the PR. I am planning on merging our branch as a foundation for continued work.

Our branch contains the entirety of this PR (as of last week) with some changes on top of that. I did squash these commits, but made sure to retain attribution of the work.

Would it be ok with you if I close this PR and merge our draft branch? I have a little clean-up left to do and one more commit to roll in to the PR.

@gregmedd gregmedd closed this Jul 25, 2024
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.

Implement and test transport::ZenohUTransport
2 participants