fix pybind11 high-level binds (#1128)

Co-authored-by: Silvio Traversaro <[email protected]>
robotology · Nov 27, 2023 · 0aafd95 · 0aafd95
1 parent 96b17da
commit 0aafd95
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -12,6 +12,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 - CMake: Permit to explicitly specify Python installation directory by setting the `IDYNTREE_PYTHON_INSTALL_DIR` CMake variable (https://github.com/robotology/idyntree/pull/1124).
 
+### Fixed
+
+- Fixed compilation of pybind11 bindings (https://github.com/robotology/idyntree/pull/1128).
+
 ## [10.0.0] - 2023-10-16
 
 ### Added

diff --git a/bindings/pybind11/CMakeLists.txt b/bindings/pybind11/CMakeLists.txt
@@ -9,7 +9,8 @@ pybind11_add_module(pybind11_idyntree idyntree.cpp
 
 target_link_libraries(pybind11_idyntree PUBLIC idyntree-core
                                                idyntree-model
-                                               idyntree-modelio)
+                                               idyntree-modelio
+                                               idyntree-high-level)
 
 # The generated Python dynamic module must have the same name as the pybind11
 # module, i.e. `bindings`.

diff --git a/src/high-level/CMakeLists.txt b/src/high-level/CMakeLists.txt
@@ -11,7 +11,7 @@ SOURCE_GROUP("Header Files" FILES ${IDYNTREE_HIGH_LEVEL_HEADERS})
 
 set(libraryname idyntree-high-level)
 
-add_library(${libraryname} ${IDYNTREE_HIGH_LEVEL_SOURCES} ${IDYNTREE_HIGH_LEVEL_HEADERS} ${IDYNTREE_HIGH_LEVEL_PRIVATE_INCLUDES})
+add_library(${libraryname} ${IDYNTREE_HIGH_LEVEL_SOURCES} ${IDYNTREE_HIGH_LEVEL_HEADERS})
 add_library(iDynTree::${libraryname} ALIAS ${libraryname})
 
 target_include_directories(${libraryname} PUBLIC "$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include>"
@@ -28,8 +28,8 @@ install(TARGETS ${libraryname}
         RUNTIME DESTINATION "${CMAKE_INSTALL_BINDIR}" COMPONENT bin
         LIBRARY DESTINATION "${CMAKE_INSTALL_LIBDIR}" COMPONENT shlib
         ARCHIVE DESTINATION "${CMAKE_INSTALL_LIBDIR}" COMPONENT lib
-        PUBLIC_HEADER DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/iDynTree/
-        PRIVATE_HEADER DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/iDynTree/)
+        PUBLIC_HEADER DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/iDynTree
+        PRIVATE_HEADER DESTINATION ${CMAKE_INSTALL_INCLUDEDIR}/iDynTree/impl)
 
 set_property(GLOBAL APPEND PROPERTY ${VARS_PREFIX}_TARGETS ${libraryname})