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
MSBuildDeps handle build_requires executables #9686
MSBuildDeps handle build_requires executables #9686
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cool!
conan/tools/microsoft/msbuilddeps.py
Outdated
for dep in build_req: | ||
dep_name = dep.ref.name | ||
dep_name = dep_name.replace(".", "_") + "_build" | ||
cpp_info = DepCppInfo(dep.cpp_info) # To account for automatic component aggregation |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We could start using the new_cpp_info
here and calling aggregate_components
for that, the same for the host/test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Agree, lets try it.
However, we should probably refactor that, so we can pick a name for new_cpp_info
that will survive in the transition to 2.0
This prevents a where the MSBuildDeps generator would generate invalid entries in the property sheet. Since conan-io/conan#9686 the property sheet uses variable references and assumes the include path is relative to the package dir.
* [hdf5/all] Use a relative include path This prevents a where the MSBuildDeps generator would generate invalid entries in the property sheet. Since conan-io/conan#9686 the property sheet uses variable references and assumes the include path is relative to the package dir. * Update recipes/hdf5/all/conanfile.py Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com> Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>
* [hdf5/all] Use a relative include path This prevents a where the MSBuildDeps generator would generate invalid entries in the property sheet. Since conan-io/conan#9686 the property sheet uses variable references and assumes the include path is relative to the package dir. * Update recipes/hdf5/all/conanfile.py Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com> Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>
Changelog: Feature:
MSBuildDeps
generator learned how to handlebuild_requires
to use executables from them.Docs: conan-io/docs#2248
Close #9545