-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
rapidyaml: remove internal c4core #9061
rapidyaml: remove internal c4core #9061
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
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.
One nag, if you dont want to apply it...
Let me know and I'll approve
recipes/rapidyaml/all/conanfile.py
Outdated
@@ -88,7 +96,7 @@ def package_info(self): | |||
self.cpp_info.set_property("cmake_file_name", "ryml") | |||
self.cpp_info.set_property("cmake_target_name", "ryml::ryml") | |||
# TODO: create c4core recipe |
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.
Hate dismissing reviews but the comment is now misleading
# TODO: create c4core recipe |
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.
Sorry for my late response.
Thank you for pointing out my mistake.
I will fix it.
All green in build 4 (
|
Specify library name and version: rapidyaml/0.3.0
The current recipe in rapidyaml uses the internal c4core.
Since the c4core recipe has been merged into cci, it is time to remove the internal c4core from rapidyaml's recipe.
conan-center hook activated.