Skip to content
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

Increase minimum ensureconda version for py3.12 compatibility #597

Merged
merged 2 commits into from Feb 5, 2024
Merged

Increase minimum ensureconda version for py3.12 compatibility #597

merged 2 commits into from Feb 5, 2024

Conversation

jfrost-mo
Copy link
Contributor

@jfrost-mo jfrost-mo commented Feb 5, 2024

Prior to ensureconda v1.4.4 distutils is used, which was removed in python 3.12. This prevents conda-lock running under python newer than 3.11, so the minimum version of ensureconda is increased.

Fixes #542
Fixes #596

The fix involves switching from distutils.version to packaging.version from the packaging package, which is maintained by the pypa.

The conda-lock.yml files will need regenerating after this PR.

Prior to ensureconda v1.4.4 distutils is used, which was removed in
python 3.12. This prevents conda-lock running under python newer
than 3.11, so the minimum version of ensureconda is increased.

Fixes #542
@jfrost-mo jfrost-mo requested a review from a team as a code owner February 5, 2024 13:05
Copy link

netlify bot commented Feb 5, 2024

Deploy Preview for conda-lock ready!

Name Link
🔨 Latest commit b86ec43
🔍 Latest deploy log https://app.netlify.com/sites/conda-lock/deploys/65c0e2020c87a7000807a892
😎 Deploy Preview https://deploy-preview-597--conda-lock.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@maresb
Copy link
Contributor

maresb commented Feb 5, 2024

Thanks @jfrost-mo! Feel free to ping me if it's green and not merged.

@maresb
Copy link
Contributor

maresb commented Feb 5, 2024

Just pushed a fix for #596

@jfrost-mo
Copy link
Contributor Author

I was too slow with #598 then. You probably want to add packaging as an explicit dependency to the pyproject.toml now its directly used. I'll go ahead and do that.

@maresb
Copy link
Contributor

maresb commented Feb 5, 2024

No, it's already there, thanks though!

@maresb
Copy link
Contributor

maresb commented Feb 5, 2024

"packaging >=20.4",

@jfrost-mo
Copy link
Contributor Author

Oops, I'll revert my change then.

@jfrost-mo
Copy link
Contributor Author

@maresb CI is all passing now.

@maresb maresb merged commit 431c09b into conda:main Feb 5, 2024
10 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

conda-lock fails with micromamba and ensureconda 1.4.4 Ensureconda fails under python 3.12; Patch available
2 participants