Skip to content

Add mavros_extras support for Linux and macOS#377

Merged
traversaro merged 1 commit intoRoboStack:mainfrom
TayebAlsh:add-mavros-extras-patch
Jan 18, 2026
Merged

Add mavros_extras support for Linux and macOS#377
traversaro merged 1 commit intoRoboStack:mainfrom
TayebAlsh:add-mavros-extras-patch

Conversation

@TayebAlsh
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Contributor

mebbaid commented Jan 18, 2026

Thanks @TayebAlsh !

@Tobias-Fischer
Copy link
Copy Markdown
Contributor

Tobias-Fischer commented Jan 18, 2026

Let’s see if @traversaro is happy

@traversaro
Copy link
Copy Markdown
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