-
Notifications
You must be signed in to change notification settings - Fork 183
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
Fix cmake install #3228
Fix cmake install #3228
Changes from 3 commits
1f222ce
0ea5308
b7b7d3e
0a1ecba
9e8aa4b
e0ad9ed
9853cbb
759b80f
511584f
6568c0f
e11198b
13b350e
4035463
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,4 @@ | ||
add_library(mpiio SHARED mpiio.cpp) | ||
target_include_directories(mpiio PUBLIC ${CMAKE_CURRENT_SOURCE_DIR}) | ||
target_link_libraries(mpiio PRIVATE EspressoConfig EspressoCore MPI::MPI_CXX) | ||
install(TARGETS mpiio LIBRARY DESTINATION ${PYTHON_INSTDIR}/espressomd) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. What is the purpose of the new There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. These libraries were missing from the installation and hence made the espressomd module unusable. |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,3 +1,4 @@ | ||
add_library(pdbreader SHARED readpdb.cpp) | ||
target_link_libraries(pdbreader PUBLIC EspressoConfig EspressoCore pdbparser) | ||
target_include_directories(pdbreader PUBLIC ${CMAKE_CURRENT_SOURCE_DIR}) | ||
install(TARGETS pdbreader LIBRARY DESTINATION ${PYTHON_INSTDIR}/espressomd) |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,5 +1,6 @@ | ||
add_library(pdbparser SHARED ${CMAKE_CURRENT_SOURCE_DIR}/src/PdbParser.cpp) | ||
target_include_directories(pdbparser PUBLIC ${CMAKE_CURRENT_SOURCE_DIR}/include) | ||
install(TARGETS pdbparser LIBRARY DESTINATION ${PYTHON_INSTDIR}/espressomd) | ||
if(WITH_UNIT_TESTS) | ||
add_subdirectory(unit_tests) | ||
endif(WITH_UNIT_TESTS) |
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.
Why is SOVERSION not needed anymore?
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.
All libraries get to installed in the python site-packages directory and in there so versioning isn’t required anymore. Plus half of the new library miss it anyhow.