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

Upgrade MAVLink in dependencies.rosinstall #1995

Closed
evan-palmer opened this issue Sep 24, 2024 · 4 comments
Closed

Upgrade MAVLink in dependencies.rosinstall #1995

evan-palmer opened this issue Sep 24, 2024 · 4 comments

Comments

@evan-palmer
Copy link
Contributor

Issue details

The version of MAVLink targeted in the dependencies.rosinstall file (i.e., release/foxy/mavlink) does not include the latest changes to pymavlink needed to support Python 3.12 (available in this commit from Tridge). These changes are needed to help address #1958.

@evan-palmer
Copy link
Contributor Author

@vooon is there anything I can do to help with this? I took a look at the mavlink-gpb-release repo, and it looks like the bloom build needs to be manually triggered?

@vooon
Copy link
Member

vooon commented Sep 24, 2024

https://github.com/mavlink/mavlink-gbp-release/blob/release/noetic/mavlink/pymavlink/generator/mavgen.py

So 2024.6.6 should have it.

@vooon
Copy link
Member

vooon commented Sep 24, 2024

NOTE: this is the correct one for ROS2: https://github.com/ros2-gbp/mavlink-gbp-release

Foxy is EOLed, and no release accepted to EOLed distros.

@evan-palmer
Copy link
Contributor Author

Oh perfect! I hadn't seen that repo. I just did some testing, and things work as expected with Jazzy. Thanks!

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

No branches or pull requests

2 participants