-
Notifications
You must be signed in to change notification settings - Fork 776
Use CargoPythonBundle for polars-0.15.6-foss-2022a #24998
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
Use CargoPythonBundle for polars-0.15.6-foss-2022a #24998
Conversation
|
@boegelbot please test @ jsc-zen3 |
|
Test report by @Flamefire |
|
@Thyre: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... Details- notification for comment with ID 3709991427 processed Message to humans: this is just bookkeeping information for me, |
|
Test report by @Flamefire |
|
Test report by @boegelbot |
|
Due to not using Edit: One dependency doesn't has a |
|
Test report by @Flamefire |
|
Uff. A dependency of a dependency (crates) got updated and now requires a newer cargo version. |
|
yeah i had this back on my list in #20718 for 5.0, but it was to painful to fix this one, as there is no way to generate an old setup. also #19697 I don't remember where this discussion was, but i did suggest removing it a long time ago, but others didn't agree, preferring to wait for it to fall away as the toolchain was deprecated, as there could, theoretically, be someone using this already built as a dep for something else and that way it would still "work". I'm perfectly fine ith removing it together with infercnvpy-0.4.2-foss-2022a.eb which depends on it. |
|
@boegel What do you think? |
|
I propose removing the 2 ECs in #25066 |
(created using
eb --new-pr)(likely) requires:
CargoPythonPackagethe default class for extensions/components ofCargoPythonBundleeasybuild-easyblocks#3993The
maturinextension usescargoand hence tries to write to$HOMEWe could replace some extensions by the
maturineasyconfig but then that would not be installed anymore. We possibly could makematurina runtime dependency (even though it is not/might not be) to avoid that, but going with the minimal change. Especially as we don't have a maturin-0.14.6 EC but only newer ones