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
Fix cpp_info.name not used in cmake find generators for dependencies #5973
Conversation
|
||
class Conan(ConanFile): | ||
requires = "hello2/1.0" | ||
generators = "cmake_find_package" |
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.
This test is repeated here just using a different generator, but not sure where to place it to avoid repeating code
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.
Don't worry too much, leave it at the moment.
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.
I would say the test improvements can be done later to develop to speed up the minor release.
'exports_sources = "src/*"\n requires = "hello/1.0"', | ||
output=client.out) | ||
client.run("create .") | ||
cmakelists = """ |
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.
No need to change it now, we need to do the release. But for the future, please try to use textwrap.dedent()
for cleaner tests layout.
|
||
class Conan(ConanFile): | ||
requires = "hello2/1.0" | ||
generators = "cmake_find_package" |
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.
Don't worry too much, leave it at the moment.
Changelog: Bugfix: Fix
cpp_info.name
not used in cmake find generators for dependenciesDocs: omit
develop
branch, documenting this one.#TAGS: slow
Note: By default this PR will skip the slower tests and will use a limited set of python versions. Check here how to increase the testing level by writing some tags in the current PR body text.