Fix OpenMPI to version 4 for compatibility with most systems#296
Conversation
giordano
left a comment
There was a problem hiding this comment.
Not ideal, but don't have better options at the moment
|
Do we also want to add |
|
Uhm, no, I don't think that's needed, is it? |
|
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #296 +/- ##
=======================================
Coverage 94.64% 94.64%
=======================================
Files 9 9
Lines 654 654
=======================================
Hits 619 619
Misses 35 35 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Adding it to this environment won't do much for when people install the package. |
Following a discussion with @giordano. There are compatibility issues between OpenMPI 4 and 5 and using OpenMPI_jll@v5 with a OpenMPI4 implementation was causing errors. We expect most systems we run on to still be on OpenMPI 4 so this seems reasonable.
Ref JuliaParallel/MPI.jl#835 (comment)