-
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
CPython: Update deps (1.x only) #21426
Conversation
I detected other pull requests that are modifying cpython/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. |
@RubenRBS :) |
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.
Thanks! Now that the ncruses recipe has been merged, we can finally focus on this PR :)
I have only 1 question about the usage of the force traitm otherwise looks good!
recipes/cpython/all/conanfile.py
Outdated
self.requires("libuuid/1.0.3") | ||
self.requires("libxcrypt/4.4.25") | ||
if not is_apple_os(self): | ||
self.requires("util-linux-libuuid/2.39.2", force=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.
Checking the PR now before merging, why is force needed here?
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.
It's getting a version conflict with another dependency that uses util-linux-libuuid, in 2.x this doesn't seem to force the version? Is there a better way to do this?
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.
@Ahajha which dependency is it conflicting with? I can't see anything in the logs - perhaps you need to update your local cache?
The normal resolution to this kind of issue is to bump the util-linux-libuuid
version on the other dependency so that both are at the most recent version.
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.
Ah, looks like it was a stale cache. It came from fontconfig
, which comes from tk
. I'll remove it.
This comment has been minimized.
This comment has been minimized.
Hooks produced the following warnings for commit 30a42b0cpython/2.7.18@#3da44cec5327d6a00056a7a2e12efb8d
|
I see CPython 2.x is failing, I'll try briefly to get it working again but I don't think it's worth too much effort currently. |
We can merge this as-is on our end, no worries about that, and work on v2 compat at a later point :) |
I mean CPython 2.7.18, not Conan 2 :) I'll try briefly to see if I can find the cause, if not, I believe there was separate discussion about dropping support for it, at the very least it's low priority. It's also causing other packages to not be built, which I imagine we want. Edit: Seems to be an issue when the package is actually downloaded, rather than a CPython 2.x issue. I'm guessing this test was added since the package was created last, it seems that there's a lot of hard-coding to paths that's happening. I'll take this into account in the 2.0 migration. Until then, review away :) |
This comment has been minimized.
This comment has been minimized.
Conan v1 pipeline ❌Failure in build 5 (
Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability. Conan v2 pipeline ❌
The v2 pipeline failed. Please, review the errors and note this is required for pull requests to be merged. In case this recipe is still not ported to Conan 2.x, please, ping See details:Failure in build 5 (
Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability. |
Closing this in favor of #21387 |
Specify library name and version: cpython/all
Since I was doing a small PR anyways, I wanted to add some of the simple Conan 2.0 migration steps to ease the diff in future PRs.
Current min version is set to 1.53.0, I don't know if the current version actually needs 1.53.0, probably around 1.51, but it will need to be 1.53.0 eventually anyways.