-
Notifications
You must be signed in to change notification settings - Fork 561
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
PaPILO requires OpenBLAS #5437
PaPILO requires OpenBLAS #5437
Conversation
@matbesancon |
in that case, there seems to be a Blas dependency in PaPILO itself, hence my surprise that things didn't break before |
can we merge this? cc @giordano |
I'm not sure this is the right fix. How did libscip end up linking to libopenblas, if that wasn't in the same environment in the first place? |
through papilo, I have to check exactly where it is needed |
@giordano it seems papilo does require it: |
What is not clear to me is: how this ended up linking to openblas if openblas wasn't a (direct at least) dependency? What has changed since last build? Without clear answers to these questions I can't tell what's going on here. |
CMake includes the detection logic for BLAS libraries in its default install, so as long as the library and include headers exist in the library/include folder, CMake will happily find it even if we don't have a listed direct dependency on it. Ipopt (or another package) probably previously relied on OpenBLAS when this was last built, so it was transitively added into the right directories in the build image, and therefore the CMake logic automatically picked it up and linked it without throwing an error at configure/link time. Now that the transitive dependency is broken, it broke at runtime. |
Ok, now I see. Thanks for the recap! |
For the record: #5525 |
Resolving
JuliaRegistries/General#67137 (comment)
I'm surprised this didn't break earlier