Use conditional linking instead of dummy targets for nvImageCodec

Author: cdinea

cpp/plugins/cucim.kit.cuslide2/CMakeLists.txt

@@ -240,23 +240,13 @@ target_link_libraries(${CUCIM_PLUGIN_NAME}
             deps::pugixml
             deps::json
             deps::libdeflate
-            deps::nvimgcodec
+            $<TARGET_NAME_IF_EXISTS:deps::nvimgcodec>
         )
 
-# Add nvImageCodec compile definition if the target exists and has content
+# Add nvImageCodec compile definition if the target exists
 if(TARGET deps::nvimgcodec)
-    get_target_property(NVIMGCODEC_LOCATION deps::nvimgcodec IMPORTED_LOCATION)
-    get_target_property(NVIMGCODEC_INTERFACE_LINK deps::nvimgcodec INTERFACE_LINK_LIBRARIES)
-
-    # Check if it's a real target (has location or interface links) vs dummy target
-    if(NVIMGCODEC_LOCATION OR NVIMGCODEC_INTERFACE_LINK OR TARGET nvimgcodec::nvimgcodec)
-        target_compile_definitions(${CUCIM_PLUGIN_NAME} PRIVATE CUCIM_HAS_NVIMGCODEC)
-        message(STATUS "✓ nvImageCodec enabled - GPU-accelerated JPEG/JPEG2000 decoding available")
-    else()
-        message(STATUS "⚠ nvImageCodec target exists but is dummy - GPU acceleration disabled")
-    endif()
-else()
-    message(STATUS "⚠ nvImageCodec target not found - GPU acceleration disabled")
+    target_compile_definitions(${CUCIM_PLUGIN_NAME} PRIVATE CUCIM_HAS_NVIMGCODEC)
+    message(STATUS "✓ nvImageCodec enabled - GPU-accelerated JPEG/JPEG2000 decoding available")
 endif()
 if (TARGET CUDA::nvjpeg_static)
         target_link_libraries(${CUCIM_PLUGIN_NAME}

cpp/plugins/cucim.kit.cuslide2/cmake/deps/nvimgcodec.cmake

@@ -125,7 +125,7 @@ if (NOT TARGET deps::nvimgcodec)
             endif()
         endif()
 
-        # Create target based on detection results (single location for target creation)
+        # Create target only if nvImageCodec was found
         if(NVIMGCODEC_LIB_PATH AND EXISTS "${NVIMGCODEC_LIB_PATH}")
             add_library(deps::nvimgcodec SHARED IMPORTED GLOBAL)
             set_target_properties(deps::nvimgcodec PROPERTIES
@@ -140,8 +140,6 @@ if (NOT TARGET deps::nvimgcodec)
             set(NVIMGCODEC_LIB_PATH ${NVIMGCODEC_LIB_PATH} CACHE INTERNAL "" FORCE)
             mark_as_advanced(NVIMGCODEC_INCLUDE_PATH NVIMGCODEC_LIB_PATH)
         else()
-            # Create dummy target when not found
-            add_library(deps::nvimgcodec INTERFACE IMPORTED GLOBAL)
             message(STATUS "✗ nvImageCodec not found - GPU acceleration disabled")
             message(STATUS "To enable nvImageCodec support:")
             message(STATUS "  Option 1 (conda): conda install libnvimgcodec-dev -c conda-forge")