Skip to content

Commit

Permalink
Fix a build issue when BUILD_SHARED_LIBS=ON with the --as-needed
Browse files Browse the repository at this point in the history
linker flag (e.g. in Gentoo), as reported in issue #958.

Thanks to @markus-meier74 and @do-jason for the report and
investigation.
  • Loading branch information
biochem-fan committed Apr 6, 2023
1 parent ec417f5 commit 884ff0d
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion src/apps/CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -219,8 +219,11 @@ else()
endif(GUI)
endif()

# FFTW_LIBRARIES is used even with MKLFFT.
# This is empty for ICC.
# See https://github.com/3dem/relion/issues/958#issuecomment-1498521352.
target_link_libraries(relion_lib ${FFTW_LIBRARIES})
if(NOT MKLFFT)
target_link_libraries(relion_lib ${FFTW_LIBRARIES})
if(BUILD_OWN_FFTW)
add_dependencies(relion_lib own_fftw_lib)
endif()
Expand Down

0 comments on commit 884ff0d

Please sign in to comment.