Merge pull request #6517 from blowekamp/target_link_libraries_dcmtk

COMP: Minimize linking with DCMTK libraries

Author: blowekamp

Modules/IO/DCMTK/src/CMakeLists.txt

@@ -6,6 +6,10 @@ set(
   itkDCMTKSeriesFileNames.cxx
 )
 
+# Remove ITKDCMTK from private depends so itk_module_add_library does not link
+# all DCMTK libraries transitively; only the required subset is linked below.
+list(REMOVE_ITEM ITK_MODULE_ITKIODCMTK_PRIVATE_DEPENDS ITKDCMTK)
+
 itk_module_add_library(
   ITKIODCMTK
   ${ITKIODCMTK_SRC}
@@ -14,6 +18,17 @@ itk_module_add_library(
 # itkDCMTKFileReader.h is an internal header in this directory (not installed).
 target_include_directories(ITKIODCMTK PRIVATE ${CMAKE_CURRENT_SOURCE_DIR})
 
+target_link_libraries(
+  ITKIODCMTK
+  PRIVATE
+    dcmdata
+    dcmimgle
+    dcmimage
+    dcmjpeg
+    dcmjpls
+    oflog
+)
+
 if(DEFINED DCMTK_HAVE_CONFIG_H_OPTIONAL AND NOT DCMTK_HAVE_CONFIG_H_OPTIONAL)
   # Workaround Debian packaging issue - See FindDCMTK.cmake for more details
   target_compile_definitions(ITKIODCMTK PRIVATE ${DCMTK_DEFINITIONS})

Modules/IO/IOTransformDCMTK/src/CMakeLists.txt

@@ -4,7 +4,13 @@ set(
   itkDCMTKTransformIOFactory.cxx
 )
 
+# Remove ITKDCMTK from private depends so itk_module_add_library does not link
+# all DCMTK libraries transitively; only the required subset is linked below.
+list(REMOVE_ITEM ITK_MODULE_IOTransformDCMTK_PRIVATE_DEPENDS ITKDCMTK)
+
 itk_module_add_library(IOTransformDCMTK ${IOTransformDCMTK_SRCS})
 
+target_link_libraries(IOTransformDCMTK PRIVATE dcmdata)
+
 # itkDCMTKTransformIO.hxx is a private implementation header in this directory.
 target_include_directories(IOTransformDCMTK PRIVATE ${CMAKE_CURRENT_SOURCE_DIR})