Skip to content

Resolve build error when linking yaml-cpp#1994

Merged
vooon merged 3 commits intomavlink:ros2from
evan-palmer:bug-yaml-cpp-humble-build
Sep 22, 2024
Merged

Resolve build error when linking yaml-cpp#1994
vooon merged 3 commits intomavlink:ros2from
evan-palmer:bug-yaml-cpp-humble-build

Conversation

@evan-palmer
Copy link
Copy Markdown
Contributor

This PR addresses the pre-Jazzy build error introduced in #1988, where yaml-cpp cannot be found. I've done some testing to verify that things build and run as expected using ROS 2 Humble, Iron, Jazzy, and Rolling.

@evan-palmer evan-palmer marked this pull request as ready for review September 22, 2024 01:53
@evan-palmer
Copy link
Copy Markdown
Contributor Author

evan-palmer commented Sep 22, 2024

Hmm looking into the issue with the CI now.

EDIT: Looks like I made an error when rebasing the first time. Should be fixed now. Going to let the CI run once before I mark the PR as ready for review.

@evan-palmer
Copy link
Copy Markdown
Contributor Author

Looks like all errors are from previous tests - not the builds

@evan-palmer evan-palmer marked this pull request as ready for review September 22, 2024 04:40
Copy link
Copy Markdown
Member

@vooon vooon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

Post-Jazzy and Pre-Jazzy builds have diverged

2 participants