-
Notifications
You must be signed in to change notification settings - Fork 1.7k
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
[ranges-v3] Fix issues with ranges-v3 recipe in Conan 2.0 #14628
[ranges-v3] Fix issues with ranges-v3 recipe in Conan 2.0 #14628
Conversation
872b0c7
Co-authored-by: SpaceIm <30052553+SpaceIm@users.noreply.github.com>
@@ -67,7 +69,7 @@ def package(self): | |||
def package_info(self): | |||
self.cpp_info.components["range-v3-meta"].names["cmake_find_package"] = "meta" |
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 see that target names of components are wrong in CMakeDeps
Conan v1 pipelineAll green in build 2 (
|
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.
LGTM
Specify library name and version: ranges-v3/all
Small fixes to support Conan v2
is_msvc
to check for the Visual C++ compiler instead of testing for the compiler name, as Conan 2 will raise an error sinceVisual Studio
was removed in favour ofmsvc
self.output.warning
instead ofwarn
which is no longer in conan 2