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

[asio] Add cmake target #6751

Merged
merged 3 commits into from
Jun 4, 2019
Merged

[asio] Add cmake target #6751

merged 3 commits into from
Jun 4, 2019

Conversation

Helco
Copy link
Contributor

@Helco Helco commented Jun 4, 2019

Adds a exported cmake target for asio such that the library can be used with:

find_package(asio CONFIG REQUIRED)
target_link_libraries(main PUBLIC asio::asio)

EDIT: perfect example of why cmake targets are useful. fastrtps was trusting on a include directory variable being defined and valid.
A bit weird was that there were no error messages in the output logs

@Rastaban Rastaban merged commit e3e495d into microsoft:master Jun 4, 2019
@Helco Helco deleted the asio_target branch June 5, 2019 05:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants