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

zenoh-pico: update to correct dev/1.0.0-px4 branch which is advertised by GitHub #23492

Merged
merged 1 commit into from
Aug 6, 2024

Conversation

MaEtUgR
Copy link
Member

@MaEtUgR MaEtUgR commented Aug 5, 2024

Solved Problem

#23462 (comment)

Solution

Switch zenoh-pico submodule from
https://github.com/PX4/zenoh-pico/commits/fff1a8c168e10d4ed9cbf2aa7d67ede07f239c52
to
https://github.com/PX4/zenoh-pico/tree/dev/1.0.0-px4 (PX4/zenoh-pico@f093aa7)

Alternatives

Please check the versions in review. I'm only guessing that's what you wanted. FYI @AlexisTM

Test coverage

This submodule commit was advertised by GitHub and hence is available for me locally.

@PetervdPerk-NXP
Copy link
Member

The PX4 fork of Zenoh-pico is now up to date with Zenoh pico dev/1.0.0 which contains the socket fix.

I would propose to point it to PX4/zenoh-pico@e6cdbf7
https://github.com/PX4/zenoh-pico/tree/dev/1.0.0

@AlexisTM
Copy link
Contributor

AlexisTM commented Aug 5, 2024

That is indeed, sorry for the mess up.

@MaEtUgR
Copy link
Member Author

MaEtUgR commented Aug 6, 2024

ok, I'll do that 👍

@MaEtUgR MaEtUgR force-pushed the zenoh-pico-correct-1.0.0-branch branch from f56cbc5 to a8ac322 Compare August 6, 2024 08:35
@MaEtUgR
Copy link
Member Author

MaEtUgR commented Aug 6, 2024

CI failure that's related 😬
image

which is up to date containing "Use SO_REUSEPORT only if it exists"
and is advertised by GitHub because the commit is on a branch
@MaEtUgR MaEtUgR force-pushed the zenoh-pico-correct-1.0.0-branch branch from a8ac322 to 7f4c8f8 Compare August 6, 2024 09:37
@MaEtUgR
Copy link
Member Author

MaEtUgR commented Aug 6, 2024

Because of the build error with https://github.com/PX4/zenoh-pico/tree/dev/1.0.0 caught by CI that's related to eclipse-zenoh/zenoh-pico#563 and type casting I went back to my original suggestion referring to https://github.com/PX4/zenoh-pico/tree/dev/1.0.0-px4 instead. That will get main successfully fetch submodules again and you can look into the build issue and update to the latest 1.0.0 with more context than I have.

@MaEtUgR MaEtUgR marked this pull request as ready for review August 6, 2024 11:41
@MaEtUgR MaEtUgR merged commit 84d4ee0 into main Aug 6, 2024
91 of 95 checks passed
@MaEtUgR MaEtUgR deleted the zenoh-pico-correct-1.0.0-branch branch August 6, 2024 11:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants