-
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
openldap: migrate to Conan v2 #18800
Conversation
valgur
commented
Jul 22, 2023
•
edited
Loading
edited
- Resolves (#16749) openldap: conan v2 support #16762
I detected other pull requests that are modifying openldap/all recipe: This message is automatically generated by https://github.com/ericLemanissier/conan-center-conflicting-prs so don't hesitate to report issues/improvements there. |
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit c87d9b7openldap/2.6.1
|
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.
Hooks produced the following warnings for commit ad2962fopenldap/2.6.1
|
This pull request has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit 10c28eeopenldap/2.6.1@#23d543de15a9646b74cd04cb41007797
|
10c28ee
to
4f791be
Compare
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit 4f791beopenldap/2.6.1@#23d543de15a9646b74cd04cb41007797
|
4f791be
to
ff1a39d
Compare
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit 7d5a9feopenldap/2.6.1@#2f3a8f1e9667a58c5b03e0eee47241dd
|
Any news on this one? I require it for our Conan 2.0 migration |
7df18b2
to
845ef51
Compare
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.
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.
Thank you! Only few changes and we good to go .
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Uilian Ries <uilianries@gmail.com>
Conan v1 pipeline ✔️All green in build 3 (
Conan v2 pipeline ✔️
All green in build 3 ( |
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
@valgur I did a small change to fix the previous error, by simplifying the libexec installation. The PR is merged now, thank you! |
Thanks, @uilianries! |
* openldap: migrate to Conan v2 * openldap: clean up package_folder * openldap: bump cyrus-sasl * openldap: add VirtualRunEnv for OpenSSL * openldap: add v2.6.6, simplify patching * openldap: project is C-only, not C++ * openldap: add resolv system dep * openldap: downgrade to cyrus-sasl/2.1.27 to fix missing binary, maybe * openldap: bump cyrus-sasl * openldap: bump to v2.6.7 * openldap: apply PR suggestions * openldap: add components with pkg_config_names * Simplify slapd install for openldap Signed-off-by: Uilian Ries <uilianries@gmail.com> --------- Signed-off-by: Uilian Ries <uilianries@gmail.com> Co-authored-by: Uilian Ries <uilianries@gmail.com>