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

Create macOS platform tags using packaging #439

Merged
merged 4 commits into from
Jun 28, 2023

Conversation

jjhelmus
Copy link
Contributor

Description

Populate the list of supported platforms tags for the macOS platform using the mac_platforms function in packaging.tags. This given a complete listing of platform tags which is also used by pip.

This avoiding the need to periodically update a static listing of tags. The new MACOS_VERSION variable will need to be updated periodically to reflect the latest version of macOS.

Likely fixes #403 and #436

Populate the list of macOS platform tags using the mac_platforms
function from packaging.tags using a constant macOS version tuple.
Refactor the __init__ method of the PlatformEnv class to parse the
platform string into a (operating) system and arch[itecture].
@jjhelmus jjhelmus requested a review from a team as a code owner June 23, 2023 18:54
@netlify
Copy link

netlify bot commented Jun 23, 2023

Deploy Preview for conda-lock ready!

Name Link
🔨 Latest commit e6acc97
🔍 Latest deploy log https://app.netlify.com/sites/conda-lock/deploys/649c51356caa100008d4ccc7
😎 Deploy Preview https://deploy-preview-439--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 Jun 23, 2023

Thanks a lot for this @jjhelmus! This looks more promising than the ad-hoc suggestion in #403.

I am wondering how we should test this. Probably by adding #403 and #436 as test cases, if they are reproducible and are indeed closed by this.

@mariusvniekerk mariusvniekerk merged commit 594f48e into conda:main Jun 28, 2023
9 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.

Pip dependency won't resolve on osx-64 - RuntimeError: Unable to find installation candidates for ortools
3 participants