Skip to content
This repository was archived by the owner on Feb 15, 2023. It is now read-only.

Enable universal2 wheels #123

Closed
wants to merge 4 commits into from
Closed

Conversation

judahrand
Copy link
Contributor

Changes ported from `numpy-wheels`  found at:
- MacPython/numpy-wheels#104
- MacPython/numpy-wheels#115
@isuruf
Copy link
Contributor

isuruf commented Jun 28, 2021

You also need MacPython/numpy-wheels#109 and MacPython/numpy-wheels#110

@judahrand
Copy link
Contributor Author

You also need MacPython/numpy-wheels#109 and MacPython/numpy-wheels#110

Should be included now, thanks very much for pointing those out.

@judahrand
Copy link
Contributor Author

I believe the PR I opened on scipy is also needed for the universal2 builds?

Comment on lines +19 to +39
# TODO: Remove once https://github.com/matthew-brett/multibuild/pull/409 lands
function pyinst_fname_for_version {
# echo filename for OSX installer file given Python and minimum
# macOS versions
# Parameters
# $py_version (Python version in major.minor.extra format)
# $py_osx_ver: {major.minor | not defined}
# if defined, the minimum macOS SDK version that Python is
# built for, eg: "10.6" or "10.9", if not defined, infers
# this from $py_version using macpython_sdk_for_version
local py_version=$1
local py_osx_ver=${2:-$(macpython_sdk_for_version $py_version)}
local inst_ext=$(pyinst_ext_for_version $py_version)
echo "python-${py_version}-macosx${py_osx_ver}.${inst_ext}"
}

function install_delocate {
check_pip
$PIP_CMD install git+https://github.com/isuruf/delocate.git@arm64
}

Copy link
Contributor

Choose a reason for hiding this comment

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

If you update multibuild, this can be removed.

@rgommers
Copy link
Contributor

Something is still wrong, on TravisCI:

Could not download "https://anaconda.org/multibuild-wheels-staging/openblas-libs/v0.3.17/download/openblas-v0.3.17-manylinux1_aarch64.tar.gz"

It should be manylinux2014 instead of manylinux1

@rgommers
Copy link
Contributor

Appveyor is also unhappy:

could not find library 'quadpack' in directories ['C:\\projects\\scipy-wheels\\scipy\\build\\openblas_lapack_info']
could not find library 'mach' in directories ['C:\\projects\\scipy-wheels\\scipy\\build\\openblas_lapack_info']
could not find library 'openblas_lapack_info' in directories ['C:\\projects\\scipy-wheels\\scipy\\build\\openblas_lapack_info']
C:\mingw-w64\x86_64-6.3.0-posix-seh-rt_v5-rev1\mingw64\bin\gfortran.exe -Wall -g -Wall -g -shared -Wl,--whole-archive C:\Python39-x64\lib\openblas.a -Wl,--no-whole-archive -LC:\mingw-w64\x86_64-6.3.0-posix-seh-rt_v5-rev1\mingw64\lib\gcc\x86_64-w64-mingw32\6.3.0 -LC:\Python39-x64\libs -LC:\Python39-x64\PCbuild\amd64 -Lbuild\temp.win-amd64-3.9 -o build\temp.win-amd64-3.9\Release\.libs\libopenblas.XWYDX2IKJW2NMTWSFYNGFUWKQU3LYTCZ.gfortran-win_amd64.dll -Wl,--allow-multiple-definition -Wl,--output-def,build\temp.win-amd64-3.9\Release\libopenblas.XWYDX2IKJW2NMTWSFYNGFUWKQU3LYTCZ.gfortran-win_amd64.def -Wl,--export-all-symbols -Wl,--enable-auto-import -static -mlong-double-64
C:\Python39-x64\lib\openblas.a(cblas_xerbla.obj):xerbla.c:(.text+0xe): undefined reference to `__imp___acrt_iob_func'
...
collect2.exe: error: ld returned 1 exit status
error: Command "C:\mingw-w64\x86_64-6.3.0-posix-seh-rt_v5-rev1\mingw64\bin\gfortran.exe -Wall -g -Wall -g -shared -Wl,--whole-archive C:\Python39-x64\lib\openblas.a -Wl,--no-whole-archive -LC:\mingw-w64\x86_64-6.3.0-posix-seh-rt_v5-rev1\mingw64\lib\gcc\x86_64-w64-mingw32\6.3.0 -LC:\Python39-x64\libs -LC:\Python39-x64\PCbuild\amd64 -Lbuild\temp.win-amd64-3.9 -o build\temp.win-amd64-3.9\Release\.libs\libopenblas.XWYDX2IKJW2NMTWSFYNGFUWKQU3LYTCZ.gfortran-win_amd64.dll -Wl,--allow-multiple-definition -Wl,--output-def,build\temp.win-amd64-3.9\Release\libopenblas.XWYDX2IKJW2NMTWSFYNGFUWKQU3LYTCZ.gfortran-win_amd64.def -Wl,--export-all-symbols -Wl,--enable-auto-import -static -mlong-double-64" failed with exit status 1

@judahrand
Copy link
Contributor Author

I'll try to have a look at this this evening

@rgommers
Copy link
Contributor

Continued in gh-125, closing this PR.

@rgommers rgommers closed this Aug 23, 2021
# for free to subscribe to this conversation on GitHub. Already have an account? #.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants