Skip to content
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

Disable dlopen trick on OpenMPI >= 3.0.0 #3849

Merged
merged 2 commits into from
Aug 4, 2020
Merged

Disable dlopen trick on OpenMPI >= 3.0.0 #3849

merged 2 commits into from
Aug 4, 2020

Conversation

mkuron
Copy link
Member

@mkuron mkuron commented Aug 4, 2020

It is not needed anymore since open-mpi/ompi#3705. Related: #3812, where we document this trick.

@KaiSzuttor
Copy link
Member

find_package(MPI 3.0 REQUIRED)

Doesn't that mean that we do not need it at all?

@mkuron
Copy link
Member Author

mkuron commented Aug 4, 2020

@KaiSzuttor, you're confusing MPI (the standard) and OpenMPI (an implementation) versions. We require an MPI 3.0-compliant MPI implementation. That requirement is fulfilled for example by OpenMPI >= 1.7.5, MPICH >= 3, or MVAPICH2 >= 2.1.

@KaiSzuttor
Copy link
Member

ah you are right

@KaiSzuttor KaiSzuttor added the automerge Merge with kodiak label Aug 4, 2020
@kodiakhq kodiakhq bot merged commit 0006680 into espressomd:python Aug 4, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Merge with kodiak
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants