Skip to content

Commit

Permalink
COMP: Fix shared libs configuration for remote
Browse files Browse the repository at this point in the history
When building RTK as an ITK Remote module, the itk-module-init.cmake file
is included prior to setting the RTK_BUILD_SHARED_LIBS variable. Thus
rtkConfiguration.h was missing RTK_BUILD_SHARED_LIBS value.

Solution: Moving configuration of rtkConfiguration.h.in to ensure that all
variable are set before configuring.
  • Loading branch information
LucasGandel committed Sep 21, 2018
1 parent dd00d03 commit ab8f6cb
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 3 deletions.
4 changes: 4 additions & 0 deletions CMakeLists.txt
Original file line number Diff line number Diff line change
Expand Up @@ -119,6 +119,10 @@ if(NOT ITK_SOURCE_DIR)
include(itk-module-init.cmake)
endif()

# Propagate cmake options in a header file
configure_file(${RTK_SOURCE_DIR}/rtkConfiguration.h.in
${RTK_BINARY_DIR}/rtkConfiguration.h)

#=========================================================
# If choose to build documentation, then search for Doxygen executables.
option(RTK_BUILD_DOXYGEN "Build Doxygen Documentation" OFF)
Expand Down
3 changes: 0 additions & 3 deletions itk-module-init.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,3 @@ if(RTK_USE_CUDA)
set(RTK_CUDA_PROJECTIONS_SLAB_SIZE "16" CACHE STRING "Number of projections processed simultaneously in CUDA forward and back projections")
endif()

# Propagate cmake options in a header file
configure_file(${RTK_SOURCE_DIR}/rtkConfiguration.h.in
${RTK_BINARY_DIR}/rtkConfiguration.h)

0 comments on commit ab8f6cb

Please sign in to comment.