-
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
add lely-core/2.3.2 #15151
add lely-core/2.3.2 #15151
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
@uilianries @RubenRBS @prince-chrismc I'm really sorry for tagging and pushing, but I need to use this library in a project and would like to import it through Conan, and I think this package is ready to be merged in. Could you review it again? |
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
You can check my bot prince-chrismc/conan-center-index-pending-review#1 so learn about the status of all PRs there are fun graphs at the bottom it's currently taking ~16 days for PRs to merge so you'll need to be patient as we have a huge backlog from the holidays |
I checked that, but I became anxious (due to no review, and pressure to use it) so I am really sorry again for this. |
No worries, we understand that it's hard when you feel like your PR gets ignored, but as Chris said, it's not the case! We're just running a bit behind due to the holidays :) |
This comment has been minimized.
This comment has been minimized.
Conan v1 pipeline ✔️All green in build 37 (
|
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
* add lely * fix lint error * add description * add patch information * add patch_source * change name from lely to lely-core * fix url and homepage attributes * add test for conan v1 * link thread library in test package * include LICENSE file and remove pkgconfig and .la files * switch to explicit typing to satisfy C++11 construction * fix conanfile.py for conan v1 and fix test * specify C++ version when compiling test * fix conanfile.py for test package conan v1 * correct path of executable * remove fPIC when building shared library * specify system libraries * use rm_safe instead of del * include conanbuildinfo.cmake * correct executable path * fix v2 test * try different way of importing * force compile with gcc only * fix typo and move comment into the error message * change license to SPDX short-name form * clarify error messages * use configure_args AutotoolsToolchain instead of passing them as an argument * move configuring arguments to generate method * reuse test_package's CMake script in test_v1_package * add version 2.3.2 * remove version 2.3.1 * remove unnecessary Threads::Threads linking
Specify library name and version: lely-core/2.3.1
lely-core
is a library for interacting with CANopen and it is not available on Conan Center yet, so I want to have it available there.