-
-
Notifications
You must be signed in to change notification settings - Fork 12
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
New cctools still broken on osx-arm64 #50
Comments
Can you give a MWE? |
Here is a reproducer (from rgommers/scipy#134) on an arm64 machine:
Downgrading Given that it's in the configure stage of the build, there's nothing scipy-specific going on there - it looks like any use of |
Would you be able to reduce this down to a small Fortran/C only example so we can report it upstream? |
Yes I think so, hopefully later today (first a bunch of meetings to go to). |
The problem seemed to be an absolute path given to program main; print *, "Fortran compilation is working."; end program Adding some debug prints shows what arguments are used when trying to configure the SciPy build:
So running that manually, putting the test binary in the root of the repo, with a relative path (
Now with an absolute path:
Stripping that further down:
Showing that a simple absolute path is enough even with no other compiler arguments. |
Wow. That is a great bug report! Thank you! |
Thanks. Can you open it at https://github.com/tpoechtrager/cctools-port ? |
I assume following works for you as well right?
|
FYI, I just checked and it's not just And strangely, @isuruf is correct that copying fixes it ... so presumably you know what's going wrong?
|
Done in tpoechtrager/cctools-port#117 |
- This involves updating the PETSc submodule to 3.16.6 with a patch submitted by @fdkong See https://gitlab.com/petsc/petsc/-/merge_requests/5447 for more information - After the above update and patch, conda build fails at codesigning PETSc dylibs on Apple Silicon. There appears to have been a bug with cctools/ld64 on this platform (see conda-forge/cctools-and-ld64-feedstock#50) that might have been patched but is still experienced with the ld64 609 package used by moose-mpich 4.0.2 build_0. Downgrading to ld64 530 seems to fix this for now. - This commit also reconfigures configure_petsc.sh so that all Apple Silicon-related adjustments occur in one location. - conda/petsc/run_tests.sh was also removed since it was not used in the petsc meta.yaml testing procedure (and was superseded by a current pkg-config test)
- This involves updating the PETSc submodule to 3.16.6 with a patch submitted by @fdkong See https://gitlab.com/petsc/petsc/-/merge_requests/5447 for more information - After the above update and patch, conda build fails at codesigning PETSc dylibs on Apple Silicon. There appears to have been a bug with cctools/ld64 on this platform (see conda-forge/cctools-and-ld64-feedstock#50) that might have been patched but is still experienced with the ld64 609 package used by moose-mpich 4.0.2 build_0. Downgrading to ld64 530 seems to fix this for now. - This commit also reconfigures configure_petsc.sh so that all Apple Silicon-related adjustments occur in one location. - conda/petsc/run_tests.sh was also removed since it was not used in the petsc meta.yaml testing procedure (and was superseded by a current pkg-config test)
Issue:
this issue conda-forge/admin-requests#324 is still a problem. New builds came out and lead to unsigned binaries on osx-arm64. I don’t know the underlying cause or how to keep us from releasing updates on this.
@isuruf
@beckermr
Environment (
conda list
):Details about
conda
and system (conda info
):The text was updated successfully, but these errors were encountered: