CMakeLists.txt updates to copy the pyd file for Windows.

Author: andrestraker

examples/bindings/python/CMakeLists.txt

@@ -1,16 +1,12 @@
 cmake_minimum_required(VERSION 3.14)
 
-# IMPORTANT: Ensure the aditof target (which builds aditof.dll) is defined
-# BEFORE this copying logic runs. For example:
-# add_subdirectory(libaditof/sdk)  # (Adjust path to where aditof target is defined)
+# Ensure aditof (and optionally aditofpython) targets are defined before this file's copy logic.
 
 function(PopulateforPython targetdir)
     add_subdirectory(${targetdir})
-    # Optional: collect Python sources if you want to use them later
     file(GLOB PythonSources "${CMAKE_CURRENT_SOURCE_DIR}/${targetdir}/*.py")
 endfunction()
 
-# List of Python example directories (relative to this CMakeLists.txt)
 set(PY_EXAMPLES
     dnn
     first_frame
@@ -22,17 +18,14 @@ set(PY_EXAMPLES
     showPointCloud
 )
 
-# Unconditionally add all example subdirectories
 foreach(ex ${PY_EXAMPLES})
     PopulateforPython("${ex}")
 endforeach()
 
-# Copy .so files to each Python example directory
 if (NOT WIN32)
     if (TARGET aditof AND TARGET aditofpython)
-        # Single aggregate target that depends on both libraries
         add_custom_target(copy_so_to_python_examples ALL
-            COMMENT "Copying aditof.so and aditofpython.so to Python example directories."
+            COMMENT "Copying aditofpython shared object to Python example directories (non-Windows)."
         )
         add_dependencies(copy_so_to_python_examples aditof aditofpython)
 
@@ -44,38 +37,71 @@ if (NOT WIN32)
                 COMMAND ${CMAKE_COMMAND} -E copy_if_different
                         "$<TARGET_FILE:aditofpython>"
                         "${CMAKE_CURRENT_BINARY_DIR}/${ex}/"
-                COMMENT "Copied aditof.so and aditofpython.so to ${ex}"
+                COMMENT "Copied aditofpython to ${ex}"
             )
         endforeach()
     else()
-        message(WARNING
-            "Targets 'aditof' or 'aditofpython' not found. Define them before this point.")
+        message(WARNING "Targets 'aditof' or 'aditofpython' not found for non-Windows copy step.")
     endif()
 else()
-    # Windows: copy aditof.dll (per configuration) into each example's build dir.
+    # Windows-only copy logic.
+    # Most likely bindings live under: <binary>/libaditof/bindings/python/<CONFIG>
+    # Change this to ${CMAKE_BINARY_DIR}/build/libaditof/bindings/python if you truly have a nested build directory.
+    set(ADITOF_PY_BINDINGS_BASE "${CMAKE_BINARY_DIR}/libaditof/bindings/python")
+
+    if (CMAKE_CONFIGURATION_TYPES)
+        set(_ADITOF_PY_CFG "$<CONFIG>")
+    else()
+        if (NOT CMAKE_BUILD_TYPE)
+            message(WARNING "CMAKE_BUILD_TYPE not set; defaulting to Release.")
+            set(CMAKE_BUILD_TYPE "Release")
+        endif()
+        set(_ADITOF_PY_CFG "${CMAKE_BUILD_TYPE}")
+    endif()
+
+    set(ADITOF_PY_BINDINGS_DIR "${ADITOF_PY_BINDINGS_BASE}/${_ADITOF_PY_CFG}")
+
     if (TARGET aditof)
-        # Single aggregate target that depends on 'aditof' and performs all copies.
-        add_custom_target(copy_dll_to_examples ALL
-            COMMENT "Copying aditof.dll to Python example directories (Windows)."
+        add_custom_target(copy_runtime_to_examples ALL
+            COMMENT "Copying aditof.dll and Python bindings (${_ADITOF_PY_CFG}) to Python example directories (Windows)."
         )
-        add_dependencies(copy_dll_to_examples aditof)
+        add_dependencies(copy_runtime_to_examples aditof)
+
+        # Optional: if a target producing the Python extension exists, depend on it too.
+        if (TARGET aditofpython)
+            add_dependencies(copy_runtime_to_examples aditofpython)
+        endif()
 
         foreach(ex ${PY_EXAMPLES})
+            # Single command per example for cleaner output
             add_custom_command(
-                TARGET copy_dll_to_examples
+                TARGET copy_runtime_to_examples
                 COMMAND ${CMAKE_COMMAND} -E make_directory
                         "${CMAKE_CURRENT_BINARY_DIR}/${ex}"
                 COMMAND ${CMAKE_COMMAND} -E copy_if_different
                         "$<TARGET_FILE:aditof>"
                         "${CMAKE_CURRENT_BINARY_DIR}/${ex}/aditof.dll"
-                COMMENT "Copied aditof.dll to ${ex}"
+                # Copy the entire configuration-specific bindings directory.
+                COMMAND ${CMAKE_COMMAND} -E copy_directory
+                        "${ADITOF_PY_BINDINGS_DIR}"
+                        "${CMAKE_CURRENT_BINARY_DIR}/${ex}"
+                COMMENT "Copied aditof.dll and Python bindings to ${ex}"
             )
+
+            # ------------------------------------------------------------------
+            # Alternative (copy only the built Python extension module) - UNCOMMENT IF NEEDED:
+            # if (TARGET aditofpython)
+            #     add_custom_command(
+            #         TARGET copy_runtime_to_examples
+            #         COMMAND ${CMAKE_COMMAND} -E copy_if_different
+            #                 "$<TARGET_FILE:aditofpython>"
+            #                 "${CMAKE_CURRENT_BINARY_DIR}/${ex}/"
+            #         COMMENT "Copied aditofpython module to ${ex}"
+            #     )
+            # endif()
+            # ------------------------------------------------------------------
         endforeach()
     else()
-        message(WARNING
-            "Target 'aditof' not found. Define it before this point or add a fallback path.")
+        message(WARNING "Target 'aditof' not found. Cannot copy runtime/bindings.")
     endif()
-endif()
-
-# (Optional) Previous non-Windows Python install step removed since you stated Windows-only need.
-# If you still need install logic for other platforms, re-add it here.
+endif()