-
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
[boost] Bump dependencies #12024
[boost] Bump dependencies #12024
Conversation
Signed-off-by: Uilian Ries <uilianries@gmail.com>
Signed-off-by: Uilian Ries <uilianries@gmail.com>
Signed-off-by: Uilian Ries <uilianries@gmail.com>
I detected other pull requests that are modifying boost/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. |
Signed-off-by: Uilian Ries <uilianries@gmail.com>
Signed-off-by: Uilian Ries <uilianries@gmail.com>
This comment has been minimized.
This comment has been minimized.
Signed-off-by: Uilian Ries <uilianries@gmail.com>
All green in build 7 (
|
Hooks produced the following warnings for commit 5cef0dcboost/1.78.0
boost/1.77.0
|
Finally passed! |
see the comment I've left here, I'm relatively sure this broke the recipe: c6cddd3#r80406005 |
Hi! Looks like this pull request (I guess upgrading b2) has broken boost builds on Windows with option For example calling this
|
@SergeyKrivohatskiy thank you for reporting. Please, Open an issue (bug report) and fill all details asked on the template: https://github.com/conan-io/conan-center-index/issues/new/choose So we can reproduce your case. |
pyversion = Version(self._python_version) | ||
libformatdata["py_major"] = pyversion.major | ||
libformatdata["py_minor"] = pyversion.minor |
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.
This switches the code from using conans.client.tools.version.Version
which wraps semver
and has .major and .minor properties to conans.model.version.Version
(which has .major and .minor methods).
So the later calls to .major and .minor aren't calling the method and we get:
@(Pdb) Version(self._python_version).major
<bound method Version.major of '3.10'>
Should I revert the change or update these to
pyversion.major(fill=False)
pyversion.minor(fill=False)
@uilianries wanted to check your intention before raising a bug.
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.
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.
@markferry The intention is only one: Move to Conan 2.0 https://docs.conan.io/en/latest/conan_v2.html Of course, it's a side-effect. Thanks for sharing, it deserves a new hotfix.
…ersion This switches the code from using `conans.client.tools.version.Version` (which wraps semver and has .major and .minor properties) to `conans.model.version.Version` (which has .major and .minor methods). So the later calls to .major and .minor weren't calling the methods and we get: ```python @(Pdb) Version(self._python_version).major <bound method Version.major of '3.10'> ```
b2
andlibiconv
to the latest version available on Conan Center