Skip to content

update gfortran-install submodule#91

Merged
mattip merged 2 commits intoMacPython:masterfrom
mattip:gfortran3
Nov 9, 2022
Merged

update gfortran-install submodule#91
mattip merged 2 commits intoMacPython:masterfrom
mattip:gfortran3

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
@mattip mattip deleted the gfortran3 branch December 22, 2025 06:12
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