-
Notifications
You must be signed in to change notification settings - Fork 113
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
Use the same depends solver for nrn and crnrn #2864
Conversation
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
✔️ ff8a79b -> Azure artifacts URL |
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
This comment has been minimized.
✔️ 9e13cb4 -> Azure artifacts URL |
This comment has been minimized.
This comment has been minimized.
✔️ af4bd6b -> Azure artifacts URL |
Quality Gate passedIssues Measures |
@@ -492,7 +492,7 @@ endforeach() | |||
|
|||
# we can link to MPI libraries in non-dynamic-mpi build | |||
if(NRN_ENABLE_MPI AND NOT NRN_ENABLE_MPI_DYNAMIC) | |||
target_link_libraries(coreneuron-core PUBLIC ${MPI_CXX_LIBRARIES}) | |||
target_link_libraries(coreneuron-core PUBLIC ${MPI_C_LIBRARIES}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I seem to remember that it sometimes matters. When compiling as a C++ project, some implementations of MPI (I think OpenMPI on Ubuntu (or ArchLinux)) would rely on C++ functionality, even if we were only using the C API ourselves.
I think that's why we have:
https://github.com/BlueBrain/HighFive/blob/d51d6cc0c87737e5e2d68adfe297576161eae1a2/cmake/HighFiveConfig.cmake#L13-L16
Pull request was closed
No description provided.