-
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
libvault: add 0.59.0 #21340
libvault: add 0.59.0 #21340
Conversation
This comment has been minimized.
This comment has been minimized.
I detected other pull requests that are modifying libvault/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. |
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 for your first contribution! This PR should be built in few hours after #21399 being merged.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Uilian Ries <[email protected]>
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
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Uilian Ries <[email protected]>
Signed-off-by: Uilian Ries <[email protected]>
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Uilian Ries <[email protected]>
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Uilian Ries <[email protected]>
Conan v1 pipeline ✔️All green in build 5 (
Conan v2 pipeline ✔️
All green in build 5 (
|
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.
Thanks a lot! Looks great now, I oinly have a minor question :)
@@ -49,29 +61,18 @@ def layout(self): | |||
|
|||
def requirements(self): | |||
# public header VaultClient.h includes curl/curl.h and use several functions | |||
self.requires("libcurl/[>=7.78.0 <9]", transitive_headers=True, transitive_libs=True) | |||
self.requires("libcurl/[>=7.78.0 <9]", transitive_headers=True) |
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.
Just a minor question, could you give me a bit more insight into why this was removed? Wy is it not needed?
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 think it's because of this abedra/libvault#27 So, there is no need to make it visible downstream anymore.
* libvault: add 0.59.0 * Fix scm version import Signed-off-by: Uilian Ries <[email protected]> * Fix libvault patches Signed-off-by: Uilian Ries <[email protected]> * match the upstream master Signed-off-by: Uilian Ries <[email protected]> * apply patch on the fly Signed-off-by: Uilian Ries <[email protected]> * Add hotfix link Signed-off-by: Uilian Ries <[email protected]> * Drop version 0.48 Signed-off-by: Uilian Ries <[email protected]> * Drop old versions due Windows linkage error Signed-off-by: Uilian Ries <[email protected]> --------- Signed-off-by: Uilian Ries <[email protected]> Co-authored-by: Uilian Ries <[email protected]>
Specify library name and version: libvault/0.59.0
Adding version 0.59.0. It looks like this package was created by someone else but the versions haven't been maintained. I'll try to make sure these are updated as I create new releases.