-
-
Notifications
You must be signed in to change notification settings - Fork 17
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
Regression in 3.14.16=*_7 builds on osx-arm64 and osx-64 #114
Comments
See https://github.com/traversaro/test-import-conda-forge-packages/actions/runs/11056530360 for an example of failures. fyi @minrk |
Arg, I'll check. Looks like maybe cyipopt is also linking directly to scotch? Not sure why or how that would be, but I can investigate. |
If you want to mark build 7 broken, that can take some pressure off (or just revert it to make build 8 before trying again) |
It happens also with iDynTree, and I do not think there is any direct link to |
OK, then maybe it's something in mumps itself that needs to put scotch in run_exports. That seems weird, though. |
Done: #115 . |
I would like to help but at the moment I do not have access to a macOS machine. |
I think this error was there before these fixes on this feedstock. I saw it as soon as the new scotch version was merged: conda-forge/scotch-feedstock#90 (comment). |
I have a mac, and I think I can see the problem. It's in scotch itself, which is missing a link to libscotcherr:
I think build 7 of this package is fine, but scotch 7.0.5 is not. |
ok, I think it's actually mumps that's wrong, and it's a combination of conda-forge's default
So I see two fixes:
I'm inclined to follow debian, as we have been doing for years, since it's a single fix that solves the problem everywhere. Even petsc links ptscotcherr rather than taking scotch up on redefining SCOTCH_errorPrint, and if any package would take the opportunity to customize error handling, I would expect it to be petsc. |
Wow, thanks for the detailed explanation. I totally agree with option 1. |
By the way, the workaround of the new |
conda-forge/scotch-feedstock#92 should have the fix for this. |
Great, thanks! |
Can confirm build 7 works on mac-arm with scotch 7.0.5 build 1: test output
|
Great, I can confirm it also works on |
Probably we can close this now? |
yeah, I marked #121 as closing this, since it restores the removal (so build 10 will be identical to build 7) |
Ok! |
Solution to issue cannot be found in the documentation.
Issue
It seems that somehow #112 reintroduced the issue fixed by #111, i.e. import
idyntree
orcyipopt
(both Python libraries that depend on ipopt) fail with:Installed packages
Environment info
.
The text was updated successfully, but these errors were encountered: