-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
libssh: add recipe #23230
libssh: add recipe #23230
Conversation
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, should only add a missing exported target alias. Thanks!
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.
Added a new option for "with_symbol_versioning" and matched it to the default behaviour for libssh. Defaults to being enabled as per: Disabled if not Unix. I've implemented this as the option is deleted on Windows builds, is this the correct thing to do? |
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.
Looking good, only few details
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.
Conan v1 pipeline ✔️All green in build 9 (
|
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
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 just checked the upstream and they are actually using CMake targets in the master branch. So it should not be needed in the future.
* libssh: add recipe Related: conan-io#13449 Related: conan-io#16899 Fixes: conan-io#16747 * libssh: add option for with_symbol_versioning instead of forcing it off * libssh: add reference links for target alias definitions * libssh: remove cmake_find_package generators * Add TODO info --------- Co-authored-by: Uilian Ries <uilianries@gmail.com>
* libssh: add recipe Related: conan-io#13449 Related: conan-io#16899 Fixes: conan-io#16747 * libssh: add option for with_symbol_versioning instead of forcing it off * libssh: add reference links for target alias definitions * libssh: remove cmake_find_package generators * Add TODO info --------- Co-authored-by: Uilian Ries <uilianries@gmail.com>
Related: #13449
Related: #16899
Fixes: #16747
Specify library name and version: libssh/0.10.6
Changed replace_in_file from previous PR with apply_conandata_patches
Added
tc.cache_variables["CMAKE_TRY_COMPILE_CONFIGURATION"]
to support Visual Studio / Ninja Multi-Config generators