-
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
libest: migrate to Conan v2 #18947
libest: migrate to Conan v2 #18947
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 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. |
7eaa9e5
to
fdc44a1
Compare
fdc44a1
to
f9b52c9
Compare
This comment has been minimized.
This comment has been minimized.
To hopefully fix configure: error: cannot run C compiled programs.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
ba9213a
to
d134df2
Compare
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
11d54da
to
6a75288
Compare
This comment has been minimized.
This comment has been minimized.
6a75288
to
46790ec
Compare
This comment has been minimized.
This comment has been minimized.
|
||
def requirements(self): | ||
self.requires("openssl/1.1.1q") | ||
self.requires("openssl/1.1.1w", transitive_headers=True, transitive_libs=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.
self.requires("openssl/1.1.1w", transitive_headers=True, transitive_libs=True) | |
self.requires("openssl/[>=1.1 <4]", transitive_headers=True, transitive_libs=True) |
In case it does not support OpenSSL 3.x, you still can use range, but [>=1.1 <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.
Thanks! Using <2
did not work for some reason.
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.
@uilianries Yeah, this fails on Conan v1 for some reason. Quite annoying.
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.
What error?
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.
ERROR: Version range '>=1.1 <3' from requirement 'openssl/[>=1.1 <3]' required by 'libest/3.2.0' could not be resolved in remote 'conan-center'
Only in Conan v1.
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, I'll take a look.
Co-authored-by: Uilian Ries <uilianries@gmail.com>
This comment has been minimized.
This comment has been minimized.
Conan v1 pipeline ✔️All green in build 13 (
Conan v2 pipeline ✔️
All green in build 12 (
|
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.
Conan 1.x can not solve ranged versions using the format 1.1.1(suffix).
No description provided.