Skip to content

Commit ff37893

Browse files
eminyousvineetbansal
authored andcommitted
Fix CMake configuration for bindings source path (#207)
1 parent 8135d81 commit ff37893

2 files changed

Lines changed: 2 additions & 5 deletions

File tree

.gitignore

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -126,6 +126,3 @@ scratch/
126126

127127
# setuptools-scm managed version file
128128
src/osqp/_version.py
129-
130-
# files that are modified in the build process
131-
src/osqp/bindings.cpp

CMakeLists.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,8 +37,8 @@ list(POP_BACK CMAKE_MESSAGE_INDENT)
3737
FetchContent_MakeAvailable(osqp)
3838

3939
configure_file(${CMAKE_CURRENT_SOURCE_DIR}/src/bindings.cpp.in
40-
${CMAKE_CURRENT_SOURCE_DIR}/src/bindings.cpp)
41-
pybind11_add_module(${OSQP_EXT_MODULE_NAME} src/bindings.cpp)
40+
${CMAKE_CURRENT_BINARY_DIR}/src/bindings.cpp)
41+
pybind11_add_module(${OSQP_EXT_MODULE_NAME} ${CMAKE_CURRENT_BINARY_DIR}/src/bindings.cpp)
4242
install(TARGETS ${OSQP_EXT_MODULE_NAME} DESTINATION . COMPONENT python)
4343

4444
# TODO: We shouldn't have to do this once the interfaces are set up correctly

0 commit comments

Comments
 (0)