These should not be necessary anymore

Author: Grufoony

.github/workflows/binding.yml

@@ -32,7 +32,7 @@ jobs:
       if: matrix.os == 'ubuntu-latest'
       run: |
         sudo apt update
-        sudo apt install -y cmake build-essential libtbb-dev libspdlog-dev libsimdjson-dev doxygen
+        sudo apt install -y cmake build-essential doxygen
 
     - name: Install system dependencies (macOS)
       if: matrix.os == 'macos-latest'
@@ -189,7 +189,7 @@ jobs:
     - name: Install system dependencies
       run: |
         sudo apt update
-        sudo apt install -y cmake build-essential libtbb-dev libspdlog-dev libsimdjson-dev doxygen
+        sudo apt install -y cmake build-essential doxygen
 
     - name: Install development dependencies
       run: |

CMakeLists.txt

@@ -39,6 +39,7 @@ if(NOT CMAKE_BUILD_TYPE)
 endif()
 if(BUILD_PYTHON_BINDINGS)
   set(DSF_BUILD_PIC ON)
+  set(CMAKE_POSITION_INDEPENDENT_CODE ON)
 endif()
 
 # Set the C++ standard
@@ -133,6 +134,8 @@ endif()
 set(TBB_TEST OFF CACHE BOOL "Disable TBB tests" FORCE)
 set(TBB_EXAMPLES OFF CACHE BOOL "Disable TBB examples" FORCE)
 set(TBB_STRICT OFF CACHE BOOL "Disable TBB strict mode" FORCE)
+# set(TBB_BUILD_SHARED ON CACHE BOOL "Build TBB as shared library" FORCE)
+# set(TBB_BUILD_TBBMALLOC OFF CACHE BOOL "Disable TBB malloc" FORCE)
 
 FetchContent_Declare(
   tbb
@@ -225,6 +228,11 @@ if(BUILD_PYTHON_BINDINGS)
       PROPERTIES BUILD_WITH_INSTALL_RPATH TRUE
                  INSTALL_RPATH "${HOMEBREW_LIB}"
                  INSTALL_RPATH_USE_LINK_PATH TRUE)
+  elseif(UNIX)
+    set_target_properties(
+      dsf_python_module
+      PROPERTIES BUILD_WITH_INSTALL_RPATH TRUE
+                 INSTALL_RPATH "$ORIGIN")
   endif()
 endif()
 

setup.py

@@ -123,6 +123,32 @@ def build_extension(self, ext: CMakeExtension):
             cwd=build_temp,
         )
 
+        # Copy TBB shared library if it exists (Linux only)
+        if platform.system() == "Linux":
+            print(f"Searching for TBB shared libraries in {build_temp}...")
+            # Look for libtbb.so* recursively
+            tbb_libs = list(build_temp.glob("**/libtbb.so*"))
+            # Also look for libtbb_debug.so* if we are in debug mode or if that's what was built
+            tbb_libs.extend(list(build_temp.glob("**/libtbb_debug.so*")))
+            
+            if tbb_libs:
+                print(f"Found TBB libraries: {tbb_libs}")
+                for lib in tbb_libs:
+                    # We only want the real shared object, not symlinks if possible, 
+                    # but copying everything matching the pattern is safer to ensure we get the versioned one.
+                    # However, we need to be careful not to overwrite if multiple matches found.
+                    # Usually we want the one that the linker linked against.
+                    # Since we set RPATH to $ORIGIN, we need the library in the same dir as the extension.
+                    
+                    # Avoid copying if it's a symlink pointing to something we already copied?
+                    # simpler: just copy all of them.
+                    dest = extdir / lib.name
+                    if not dest.exists():
+                        shutil.copy2(lib, dest)
+                        print(f"Copied {lib} to {dest}")
+            else:
+                print("Warning: No TBB shared libraries found to copy.")
+
     def pre_build(self):
         """Extracts doxygen documentation from XML files and creates a C++ unordered_map"""