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

Update to python 3.12 #603

Merged
merged 11 commits into from
Jul 24, 2024
Merged

Conversation

hmaarrfk
Copy link
Contributor

@hmaarrfk hmaarrfk commented Jun 22, 2024

Closes #598
Closes #604

Checklist

  • Used a personal fork of the feedstock to propose changes
  • Bumped the build number (if the version is unchanged)
  • Reset the build number to 0 (if the version changed)
  • Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
  • Ensured the license file is being packaged.

@hmaarrfk hmaarrfk force-pushed the update_python_version_202406 branch from 3bceb51 to bddad0b Compare June 22, 2024 16:33
@hmaarrfk hmaarrfk mentioned this pull request Jun 23, 2024
5 tasks
@hmaarrfk hmaarrfk marked this pull request as ready for review June 23, 2024 01:51
@hmaarrfk hmaarrfk requested a review from a team as a code owner June 23, 2024 01:51
README.md Outdated
| Windows | x86_64 | [Miniforge3-Windows-x86_64](https://github.com/conda-forge/miniforge/releases/latest/download/Miniforge3-Windows-x86_64.exe) |
| OS | Architecture | Minimum Version | Download |
| --------|-------------------------------|------------------|-----------|
| Linux | x86_64 (amd64) | glibc >= 2.12 | [Miniforge3-Linux-x86_64](https://github.com/conda-forge/miniforge/releases/latest/download/Miniforge3-Linux-x86_64.sh) |
Copy link
Member

@mbargull mbargull Jun 23, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Instead, we can just go with 2.17 here, right?! We'll drop 2.12 in a week

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I just went with 2.17 -- otherwise, I'd expect this to get outdated quickly.

| Linux | ppc64le (POWER8/9) | glibc >= 2.17 | [Miniforge3-Linux-ppc64le](https://github.com/conda-forge/miniforge/releases/latest/download/Miniforge3-Linux-ppc64le.sh) |
| OS X | x86_64 | macOS >= 10.13 | [Miniforge3-MacOSX-x86_64](https://github.com/conda-forge/miniforge/releases/latest/download/Miniforge3-MacOSX-x86_64.sh) |
| OS X | arm64 (Apple Silicon) `(***)` | macOS >= 11.0 | [Miniforge3-MacOSX-arm64](https://github.com/conda-forge/miniforge/releases/latest/download/Miniforge3-MacOSX-arm64.sh) |
| Windows | x86_64 | Windows >= 7 | [Miniforge3-Windows-x86_64](https://github.com/conda-forge/miniforge/releases/latest/download/Miniforge3-Windows-x86_64.exe) |
Copy link
Member

@mbargull mbargull Jun 23, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@isuruf, do you know if this is still accurate or do we have things that bump this to 10?

@mbargull mbargull force-pushed the update_python_version_202406 branch from 00b8307 to eabae5e Compare June 23, 2024 09:53
@mbargull
Copy link
Member

Let's get some approvals from folks that can confirm that #598 (comment) is satisfied and merge/build this at the beginning/middle of the week to not cause downstream troubles over weekends.

@hmaarrfk
Copy link
Contributor Author

I would like to document the last known working version:

  • OSX 10.9
  • Linux Glibc 2.12

scripts/build.sh Outdated
Comment on lines 59 to 60
elif [[ "${TARGET_PLATFORM}" == linux-* ]]; then
export CONDA_OVERRIDE_OSX=10.9
Copy link
Member

@mbargull mbargull Jun 24, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
elif [[ "${TARGET_PLATFORM}" == linux-* ]]; then
export CONDA_OVERRIDE_OSX=10.9
elif [[ "${TARGET_PLATFORM}" == osx-64 ]]; then
export CONDA_OVERRIDE_OSX=10.9
elif [[ "${TARGET_PLATFORM}" == osx-arm64 ]]; then
export CONDA_OVERRIDE_OSX=11.0

I'm fine with that change.
I generally makes sense to but the overrides in.
As for 10.9 concretely, I've also thought about doing that but didn't consider it too be of relevance -- what's the reasoning for going with that still (in light that we changed the default nearly 2 months ago)?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It was to see if this propagated. Apparently not.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

if this does nothing, should we remove it entirely?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It does work; there was just a typo in the initial commit (linux-* instead of osx-*).
Even makes the PyPy builds fail -- haven't looked into why yet.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

conda=24.1.1 onward has dep pypy3.9 >=7.3.15 and that one has been built against 10.13 already.
So, we need target 10.13 for that one.

refs:
https://github.com/conda-forge/pypy3.6-feedstock/pull/116/files#diff-92d4214334555338c6e098e1462e4dec3ca2032a9f2973bf5558d6b8c00503ceR2

scripts/build.sh Outdated Show resolved Hide resolved
PyPy builds of conda>=24.1.1 depend on pypy3.9>=7.3.15 with the latter targeting macOS>=10.13.
scripts/build.sh Outdated Show resolved Hide resolved
@hmaarrfk
Copy link
Contributor Author

hmaarrfk commented Jul 2, 2024

I can't approve my own merge request, but looks good to me!

Copy link
Member

@xhochy xhochy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@jaimergp
Copy link
Member

Anything blocking the merge here? Should we just proceed?

| Linux | aarch64 (arm64) `(**)` | glibc >= 2.17 | [Miniforge3-Linux-aarch64](https://github.com/conda-forge/miniforge/releases/latest/download/Miniforge3-Linux-aarch64.sh) |
| Linux | ppc64le (POWER8/9) | glibc >= 2.17 | [Miniforge3-Linux-ppc64le](https://github.com/conda-forge/miniforge/releases/latest/download/Miniforge3-Linux-ppc64le.sh) |
| OS X | x86_64 | macOS >= 10.13 | [Miniforge3-MacOSX-x86_64](https://github.com/conda-forge/miniforge/releases/latest/download/Miniforge3-MacOSX-x86_64.sh) |
| OS X | arm64 (Apple Silicon) `(***)` | macOS >= 11.0 | [Miniforge3-MacOSX-arm64](https://github.com/conda-forge/miniforge/releases/latest/download/Miniforge3-MacOSX-arm64.sh) |
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
| OS X | arm64 (Apple Silicon) `(***)` | macOS >= 11.0 | [Miniforge3-MacOSX-arm64](https://github.com/conda-forge/miniforge/releases/latest/download/Miniforge3-MacOSX-arm64.sh) |
| OS X | arm64 (Apple Silicon) | macOS >= 11.0 | [Miniforge3-MacOSX-arm64](https://github.com/conda-forge/miniforge/releases/latest/download/Miniforge3-MacOSX-arm64.sh) |

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we still need that footnote?

(***) Apple silicon builds are experimental and haven't had testing like the other platforms.

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't have CI for macOS ARM

That said, maybe we can relax this note a bit

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm not opposed to this, but I think a new PR is warranted if we want to review the language around OSX + arm.

Lets start the builds for this as they take a while already.

@jaimergp jaimergp linked an issue Jul 24, 2024 that may be closed by this pull request
@hmaarrfk hmaarrfk merged commit cd78df0 into conda-forge:main Jul 24, 2024
24 checks passed
@EwoutH
Copy link
Contributor

EwoutH commented Aug 3, 2024

Thanks for this! When can we expect a new release which includes this PR?

@hmaarrfk
Copy link
Contributor Author

hmaarrfk commented Aug 3, 2024

We are blocked on

#614

you can for your CI purposes just create a new test environment with python 3.12. It will only cost about 100MB of downloads.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
7 participants