-
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
crossdb: add recipe #25704
crossdb: add recipe #25704
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.
Thanks! Minor comment about the test_package, otherwise I think this looks good :)
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit 7ee3722crossdb/0.9.0@#cef295ebbc2881be727168c5c5e8e159
|
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.
Thanks!
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit e8c3d17crossdb/0.9.0@#f4d391b86d58306b856cc7f3ed30f768
|
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.
LGTM.
@toge I did few changes simplifying the test package and using fix_apple_shared_install_name instead.
The project looks really simple and is pure-c. Thank you for your PR.
The Conan 1.x shows as failed, but is actually running. Should report the build result in ~10min. |
Conan v1 pipeline ✔️Warning Conan Center will stop receiving updates for Conan 1.x packages soon - please see announcement. All green in build 11 (
Conan v2 pipeline ✔️
All green in build 11 (
|
@uilianries |
* crossdb: add recipe * drop MSVC support * simplify test_package.c * Update recipes/crossdb/all/conanfile.py * make rpath on macOS * Simplify test package * Avoid emojis in the recipe. * Use fix_apple_shared_install_name to fix rpath * Do not enforce CMAKE_MACOSX_RPATH * Import fix_apple_shared_install_name --------- Co-authored-by: Abril Rincón Blanco <git@rinconblanco.es> Co-authored-by: Uilian Ries <uilianries@gmail.com>
Summary
Changes to recipe: crossdb/0.9.0
Motivation
yet another embedded RDB library.
Details
https://github.com/crossdb-org/crossdb