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.

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

Add yaml_cpp_vendor as dependency for master_node #18

Merged
merged 4 commits into from
Apr 29, 2022
Merged

Add yaml_cpp_vendor as dependency for master_node #18

merged 4 commits into from
Apr 29, 2022

Conversation

ipa-alb
Copy link
Collaborator

@ipa-alb ipa-alb commented Apr 25, 2022 •
edited by hellantos
Loading

See #19.

@hellantos hellantos changed the title commented out the yaml_cpp vendor since it doesnt let it build on the… Find package yaml_cpp_vendor without workaround Apr 28, 2022
@hellantos hellantos mentioned this pull request Apr 28, 2022
Closed
@hellantos
Copy link
Member

@ipa-alb can you test the current version. I removed the yaml_cpp_vendor include stuff and added yaml_cpp_vendor to master_node as dependency. I hope it solves the build problem on your system, too.

@hellantos hellantos changed the title Find package yaml_cpp_vendor without workaround Add yaml_cpp_vendor as dependency for master_node Apr 28, 2022
@hellantos hellantos mentioned this pull request Apr 29, 2022
Closed
@ipa-alb ipa-alb merged commit 4458720 into master Apr 29, 2022
@ipa-alb
Copy link
Collaborator Author

ipa-alb commented Apr 29, 2022

Worked out

@hellantos hellantos deleted the ipa-alb branch May 9, 2022 07:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@hellantos hellantos hellantos 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.
2 participants
@ipa-alb @hellantos