Skip to content

Add mavros_extras support for Linux and macOS #377

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.

Sign up for GitHub

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

Jump to bottom
Merged
traversaro merged 1 commit into from
Jan 18, 2026
Merged

Add mavros_extras support for Linux and macOS #377

traversaro merged 1 commit into from
Jan 18, 2026

Conversation

@TayebAlsh
Copy link
Contributor

@TayebAlsh TayebAlsh commented Jan 17, 2026

Summary

  • Adds cross-platform patch (ros-humble-mavros-extras.patch) enabling mavros_extras to build on Linux and macOS
  • Enables the package in vinca.yaml under the not wasm32 and not win condition

Changes

File Description
patch/ros-humble-mavros-extras.patch Fixes GeographicLib discovery (CONFIG mode), library linking, and tf2 API compatibility
vinca.yaml Adds mavros_extras to the package list
pkg_additional_info.yaml Adds build_number entry for mavros_extras

Platforms

  • ✅ linux-64
  • ✅ linux-aarch64
  • ✅ osx-64
  • ✅ osx-arm64

@TayebAlsh
Add mavros_extras support for Linux and macOS
c1bdb0e 
- Add cross-platform patch for mavros_extras that fixes:
  - GeographicLib discovery using CONFIG mode for conda
  - GeographicLib library linking for mavros_extras_plugins
  - tf2 API compatibility (tf2::transformToEigen)
- Enable mavros_extras in vinca.yaml (linux-64, linux-aarch64, osx-64, osx-arm64)
- Add build_number entry in pkg_additional_info.yaml
@mebbaid
Copy link
Contributor

mebbaid commented Jan 17, 2026

enabling mavros_extras to build on Linux and macOS

my understanding was that mavros is by default limited to Linux (see https://github.com/mavlink/mavros/blob/ros2/mavros/README.md#limitations). Are you able to use these packages on mac @TayebAlsh ?

@TayebAlsh
Copy link
Contributor Author

TayebAlsh commented Jan 17, 2026 •
edited
Loading

@mebbaid Yes tested on macOS! I beilive they mentioned that becuase of mavlink not being available for macOS, but there was a patch already for mavlink: #374

@mebbaid
Copy link
Contributor

mebbaid commented Jan 18, 2026

Thanks @TayebAlsh !

@Tobias-Fischer
Copy link
Contributor

Tobias-Fischer commented Jan 18, 2026

Let’s see if @traversaro is happy

@traversaro
Copy link
Member

traversaro commented Jan 18, 2026

Let’s see if @traversaro is happy

I am, thanks!

@traversaro traversaro merged commit a852320 into RoboStack:main Jan 18, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Tobias-Fischer Tobias-Fischer Tobias-Fischer approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@TayebAlsh @mebbaid @Tobias-Fischer @traversaro