Skip to content

Commit 7f9876f

Browse files
Fix wheel builds when Eigen is fetched
Include FetchContent before the Eigen fallback path so cibuildwheel can configure on runners without a system Eigen installation. Drop the no-op pp* skip selector to keep the wheel workflow output clean.
1 parent c4a7be4 commit 7f9876f

3 files changed

Lines changed: 3 additions & 5 deletions

File tree

.github/workflows/publish.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ jobs:
5757
run: python -m cibuildwheel --output-dir wheelhouse
5858
env:
5959
CIBW_BUILD: cp310-* cp311-* cp312-* cp313-*
60-
CIBW_SKIP: pp* *-musllinux_* *-win32
60+
CIBW_SKIP: *-musllinux_* *-win32
6161
CIBW_ARCHS_LINUX: auto64
6262
CIBW_ARCHS_WINDOWS: auto64
6363
CIBW_ARCHS_MACOS: auto64

.github/workflows/wheels.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@ jobs:
5858
run: python -m cibuildwheel --output-dir wheelhouse
5959
env:
6060
CIBW_BUILD: cp310-* cp311-* cp312-* cp313-*
61-
CIBW_SKIP: pp* *-musllinux_* *-win32
61+
CIBW_SKIP: *-musllinux_* *-win32
6262
CIBW_ARCHS_LINUX: auto64
6363
CIBW_ARCHS_WINDOWS: auto64
6464
CIBW_ARCHS_MACOS: auto64

CMakeLists.txt

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@ project(
2626
)
2727

2828
include(GNUInstallDirs)
29+
include(FetchContent)
2930

3031
set(CMAKE_CXX_STANDARD 17) # Enforce C++17 as the minimum standard
3132
set(CMAKE_CXX_STANDARD_REQUIRED ON) # Enforce C++17 as the minimum standard
@@ -92,9 +93,6 @@ if (CDDP_CPP_CASADI)
9293
endif()
9394
endif()
9495

95-
# Enable FetchContent for downloading dependencies
96-
include(FetchContent)
97-
9896
# autodiff
9997
set(AUTODIFF_BUILD_TESTS OFF CACHE BOOL "Don't build autodiff tests")
10098
set(AUTODIFF_BUILD_EXAMPLES OFF CACHE BOOL "Don't build autodiff examples")

0 commit comments

Comments
 (0)