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

Python 3.12 #259

Merged
merged 5 commits into from Oct 4, 2023
Merged

Conversation

thewtex
Copy link
Member

@thewtex thewtex commented Oct 3, 2023

No description provided.

@thewtex thewtex changed the title Python 3 12 Python 3.12 Oct 3, 2023
@thewtex thewtex requested review from tbirdso and jcfr October 3, 2023 20:36
Copy link
Contributor

@tbirdso tbirdso left a comment

Choose a reason for hiding this comment

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

🚀

Comment on lines 416 to 419
#for pyversion in $LATEST_3p7; do
#install_macpython $pyversion 10.9
#install_virtualenv
#done
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove?

Copy link
Member Author

Choose a reason for hiding this comment

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

I thought it would be helpful to leave as a reminder that 3.7 is associated with 10.9.

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, I didn't catch that. Would you please add a one-line comment to that effect?

Copy link
Member Author

Choose a reason for hiding this comment

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

Added per @jcfr 's inline suggestion.

@@ -39,11 +39,11 @@ TARGET_ARCH=${TARGET_ARCH:=x64}

# Specialized manylinux image tag to use for building.
if [[ ${MANYLINUX_VERSION} == _2_28 && ${TARGET_ARCH} == x64 ]]; then
IMAGE_TAG=${IMAGE_TAG:=20230106-1aeaea0}
IMAGE_TAG=${IMAGE_TAG:=20230926-9eb419c}
elif [[ ${MANYLINUX_VERSION} == _2_28 && ${TARGET_ARCH} == aarch64 ]]; then
IMAGE_TAG=${IMAGE_TAG:=2022-11-19-1b19e81}
Copy link
Contributor

Choose a reason for hiding this comment

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

Should this one be updated as well ?

Copy link
Member Author

Choose a reason for hiding this comment

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

This one is currently separate as we work towards arm cross-compilation.

elif [[ ${MANYLINUX_VERSION} == _2_28 && ${TARGET_ARCH} == aarch64 ]]; then
IMAGE_TAG=${IMAGE_TAG:=2022-11-19-1b19e81}
elif [[ ${MANYLINUX_VERSION} == 2014 ]]; then
IMAGE_TAG=${IMAGE_TAG:=20230106-1aeaea0}
IMAGE_TAG=${IMAGE_TAG:=20230926-9eb419c}
Copy link
Contributor

Choose a reason for hiding this comment

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

For future reference, the associated dockcross changes are the following: https://github.com/dockcross/dockcross/compare/1aeaea0..9eb419c

Summary:

  • CMake 3.24.2 -> 3.27.5
  • Ninja 1.11.0 -> 1.11.1
  • Git 2.36.1 -> 2.42.0
  • Manylinux 2.28
    • Update toolset from gcc-toolset-11 to gcc-toolset-12
    • pypa/manylinux_2_28_x86_64:2022-11-19-1b19e81 -> quay.io/pypa/manylinux_2_28_x86_64:2022-11-28-5d13db4
  • Manylinux 2014
    • pypa/manylinux2014_x86_64:2022-11-28-5d13db4 -> pypa/manylinux2014_x86_64:2023-08-27-bd7ad21

Co-authored-by: Jean-Christophe Fillion-Robin <jchris.fillionr@kitware.com>
@thewtex thewtex merged commit 4936bcc into InsightSoftwareConsortium:master Oct 4, 2023
1 check passed
@thewtex thewtex deleted the python-3-12 branch October 4, 2023 02:07
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.

None yet

4 participants