From 128fdcb62e2d7b9756516aec4e6f110251234974 Mon Sep 17 00:00:00 2001
From: Esteban DUGUEPEROUX <esteban.dugueperoux@gmail.com>
Date: Thu, 22 Jan 2026 18:24:43 +0100
Subject: [PATCH] fix: Add IfcGeom as a dependency of kernels target instead of
 the inverse

---
 cmake/CMakeLists.txt               | 1 -
 src/ifcgeom/CMakeLists.txt         | 6 +++++-
 src/ifcgeom/kernels/CMakeLists.txt | 2 +-
 3 files changed, 6 insertions(+), 3 deletions(-)

diff --git a/cmake/CMakeLists.txt b/cmake/CMakeLists.txt
index 0fb45e42eb..9e88026ac7 100644
--- a/cmake/CMakeLists.txt
+++ b/cmake/CMakeLists.txt
@@ -218,7 +218,6 @@ if(BUILD_IFCGEOM AND WITH_OPENCASCADE)
     find_package(OpenCASCADE REQUIRED)
     add_definitions(-DIFOPSH_WITH_OPENCASCADE)
     set(SWIG_DEFINES ${SWIG_DEFINES} -DIFOPSH_WITH_OPENCASCADE)
-    # set(OPENCASCADE_LIBRARIES ${OpenCASCADE_LIBRARIES})
     list(APPEND GEOMETRY_KERNELS opencascade)
 endif()
 
diff --git a/src/ifcgeom/CMakeLists.txt b/src/ifcgeom/CMakeLists.txt
index b50dfce590..4bacbf2f72 100644
--- a/src/ifcgeom/CMakeLists.txt
+++ b/src/ifcgeom/CMakeLists.txt
@@ -31,12 +31,16 @@ find_package(Eigen3 REQUIRED)
 
 target_link_libraries(
     IfcGeom
-    ${kernel_libraries}
     ${mapping_libraries}
     ${CMAKE_THREAD_LIBS_INIT}
     "Eigen3::Eigen"
     ${CGAL_LIBRARIES}
 )
+
+if(WITH_OPENCASCADE)
+    target_link_libraries(IfcGeom TKernel)
+endif()
+
 if(NOT WASM_BUILD)
     target_link_libraries(IfcGeom IfcParse)
 endif()
diff --git a/src/ifcgeom/kernels/CMakeLists.txt b/src/ifcgeom/kernels/CMakeLists.txt
index 5cbbd32c0d..fa9656b782 100644
--- a/src/ifcgeom/kernels/CMakeLists.txt
+++ b/src/ifcgeom/kernels/CMakeLists.txt
@@ -14,7 +14,7 @@ foreach(kernel ${GEOMETRY_KERNELS})
         PROPERTIES COMPILE_FLAGS "-DIFC_GEOM_EXPORTS" PUBLIC_HEADER "${IFCGEOM_H_FILES}"
     )
     list(APPEND kernel_libraries ${KERNEL_TARGET})
-    target_link_libraries(${KERNEL_TARGET} ${${KERNEL_UPPER}_LIBRARIES} Eigen3::Eigen)
+    target_link_libraries(${KERNEL_TARGET} ${${KERNEL_UPPER}_LIBRARIES} IfcGeom Eigen3::Eigen)
     install(
         TARGETS ${KERNEL_TARGET}
         EXPORT ${IFCOPENSHELL_EXPORT_TARGETS}