Skip to content

Fix manylinux wheel build: expose fetched Eigen3 package to autodiff #170

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
astomodynamics merged 2 commits into from
Apr 4, 2026
Merged

Fix manylinux wheel build: expose fetched Eigen3 package to autodiff #170

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
be51f01
Initial plan
Copilot Apr 4, 2026
aeb73ca
Fix wheel build: expose Eigen3 package dir to autodiff via Eigen3_DIR
Copilot Apr 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
25 changes: 16 additions & 9 deletions CMakeLists.txt
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -62,18 +62,29 @@ else()
endif()

# Find packages
find_package(Eigen3 QUIET)
find_package(Eigen3 3.4 QUIET NO_MODULE)

# If Eigen3 is not found, download it
# If Eigen3 is not found, fetch it and expose a proper Eigen3::Eigen target
if(NOT Eigen3_FOUND)
message(STATUS "Eigen3 not found. Downloading...")
FetchContent_Declare(
eigen
GIT_REPOSITORY https://gitlab.com/libeigen/eigen.git
GIT_TAG 3.4.0 # Use a stable version
GIT_TAG 3.4.0
)
FetchContent_MakeAvailable(eigen)
set(EIGEN3_INCLUDE_DIRS ${eigen_SOURCE_DIR})

# Point Eigen3_DIR to the build directory where Eigen3Config.cmake is generated
# so that downstream find_package(Eigen3) calls (e.g., from autodiff) can find it
set(Eigen3_DIR "${eigen_BINARY_DIR}" CACHE PATH "Path to Eigen3Config.cmake" FORCE)

# Ensure Eigen3::Eigen target exists for direct usage
if(NOT TARGET Eigen3::Eigen)
add_library(Eigen3::Eigen INTERFACE IMPORTED)
set_target_properties(Eigen3::Eigen PROPERTIES
INTERFACE_INCLUDE_DIRECTORIES "${eigen_SOURCE_DIR}"
)
endif()
endif()

if (CDDP_CPP_CASADI)
Expand Down Expand Up @@ -173,14 +184,10 @@ set_target_properties(${PROJECT_NAME} PROPERTIES POSITION_INDEPENDENT_CODE ON)

target_link_libraries(${PROJECT_NAME}
PUBLIC
$<IF:$<BOOL:${Eigen3_FOUND}>,Eigen3::Eigen,>
Eigen3::Eigen
autodiff
)

if(NOT Eigen3_FOUND)
target_include_directories(${PROJECT_NAME} PUBLIC ${EIGEN3_INCLUDE_DIRS})
endif()

target_include_directories(${PROJECT_NAME} PUBLIC
$<BUILD_INTERFACE:${CMAKE_CURRENT_SOURCE_DIR}/include/cddp-cpp>
$<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}>
Expand Down
Loading