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

Fix osx-arm64 builds failing on missing openmp pin #183

Closed
h-vetinari opened this issue Aug 6, 2021 · 5 comments
Closed

Fix osx-arm64 builds failing on missing openmp pin #183

h-vetinari opened this issue Aug 6, 2021 · 5 comments

Comments

@h-vetinari
Copy link
Member

This happens since the openmp 12.0.1 builds, with

   File "/Users/runner/miniforge3/conda-bld/scipy_1628206424706/_build_env/venv/lib/python3.8/site-packages/omp/__init__.py", line 107, in init_not_msvc
    self.libomp = ctypes.CDLL(libgomp_path)
  File "/Users/runner/miniforge3/conda-bld/scipy_1628206424706/_build_env/lib/python3.8/ctypes/__init__.py", line 373, in __init__
    self._handle = _dlopen(self._name, mode)
OSError: dlopen(/Users/runner/miniforge3/conda-bld/[...]/bin/../lib/libgomp.dylib, 6): no suitable image found.  Did find:
    /Users/runner/miniforge3/conda-bld/[...]/bin/../lib/libgomp.dylib: mach-o, but wrong architecture
    /Users/runner/miniforge3/conda-bld/[...]/lib/libomp.dylib: mach-o, but wrong architecture
Preparing wheel metadata: finished with status 'error'

Additionally, the pin from conda-forge/conda-forge-pinning-feedstock#1727 isn't taking here, because scipy does not depend on llvm-openmp, so I guess it's coming in through a dependency that hasn't been rebuilt with the pin yet.

Should we add an llvm-openmp constraint for osx-arm here, or how best to tackle this?

@h-vetinari h-vetinari mentioned this issue Aug 6, 2021
3 tasks
@isuruf
Copy link
Member

isuruf commented Aug 6, 2021

Should we add an llvm-openmp constraint for osx-arm here, or how best to tackle this?

It doesn't matter as scipy is not linking to openmp

@isuruf
Copy link
Member

isuruf commented Aug 6, 2021

See serge-sans-paille/pythran#1868

@h-vetinari
Copy link
Member Author

Thanks!

@h-vetinari
Copy link
Member Author

Do you already want to rebuild the pythran feedstock with that patch or wait for serge-sans-paille/pythran#1868 to be merged?

@h-vetinari
Copy link
Member Author

This has been fixed by #179

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

No branches or pull requests

2 participants