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

[v1.4] cmake: copy correct mavlink directory #1931

Merged
merged 1 commit into from
Nov 22, 2022

Conversation

julianoes
Copy link
Collaborator

Hopefully, this time I get it right.

Hopefully, this time I get it right.
@eastman-bechamo
Copy link

This fixed the header install issue. Thanks a bunch!

@julianoes julianoes merged commit 163601d into v1.4 Nov 22, 2022
@julianoes julianoes deleted the pr-v1.4-fix-mavlink-header-install-for-good branch November 22, 2022 01:17
@julianoes julianoes added the bug label Nov 22, 2022
@Katawann
Copy link
Contributor

works for me ! Thank you @julianoes

@Consti10
Copy link
Contributor

Consti10 commented Jan 25, 2023

I independently came across this issue in our mavlink fork and ended up doing the same fix implemented here

aka use ${MAVLINK_HEADERS}/mavlink instead of ${MAVLINK_HEADERS}

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants