dynamicloadmpi.py is very bad
Issue #26
resolved
The code in site-packages/dolfin/importhandler/dynamicloadmpi.py
is ugly, and it's broken because it makes an assumption on the name of the MPI library (libmpi).
It breaks in common situations, e.g. when using MPICH MPI which names the libraries 'libmpich'.
Comments (4)
-
-
I tried running without this on the buildbots and it worked fine, so I guess it can be removed.
-
Maybe you could remove it then?
Johan
-
- changed status to resolved
Fixed by revision: 53a72a56c28c3e98338f9228effd8dd45ecf9694
- Log in to comment
I need to be remembered what this hack fixed? I wonder if this was Mac related, and I also wonder if it is still needed. Johannes might know more?