Skip to content

update gfortran-install submodule #91

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

Merged
merged 2 commits into from
Nov 9, 2022
Merged

Conversation

mattip
Copy link
Collaborator

@mattip mattip commented Nov 9, 2022

Use gfortran 11.3 from conda for macos x86_64 ( from MacPython/gfortran-install#11). The latest 0.3.21 macos uploads were mistakenly labeled 10.11, they are actually 10.14 instead.

@isuruf
Copy link
Contributor

isuruf commented Nov 9, 2022

You can change to 10.9 now with MacPython/gfortran-install#11

@mattip
Copy link
Collaborator Author

mattip commented Nov 9, 2022

I wonder if somehow the gfortran-install submodule should indicate it supports MACOSX_DEPLOYMENT_TARGET>=10.9

@mattip
Copy link
Collaborator Author

mattip commented Nov 9, 2022

The artifacts have different suffixes, previously the x86_64 suffix had gf_1becaaa in it

openblas-v0.3.21-macosx_10_9_x86_64-gf_c469a42.tar.gz
openblas64_-v0.3.21-macosx_10_9_x86_64-gf_c469a42.tar.gz
openblas-v0.3.21-macosx_11_0_arm64-gf_5272328.tar.gz
openblas64_-v0.3.21-macosx_11_0_arm64-gf_5272328.tar.gz

@isuruf
Copy link
Contributor

isuruf commented Nov 9, 2022

That's expected. The gfortran compiler changed and the hash of the compiler is used in the build.

@mattip
Copy link
Collaborator Author

mattip commented Nov 9, 2022

right, that is GFORTRAN_SHA from the gfortran-install repo. So this will require an update to the numpy openblas_support script in numpy/numpy#22525

@mattip
Copy link
Collaborator Author

mattip commented Nov 9, 2022

ppc64le builds are failing, like the previous ones. That is a best-effort attempt to support that platform, so not a blocker.

@mattip mattip merged commit 87c3b8a into MacPython:master Nov 9, 2022
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.

2 participants