Skip to content

Commit d0883fa

Browse files
authored
Merge pull request #149 from jcarpent/devel
Fix packaging issue on Windows
2 parents 3491178 + 5b423f6 commit d0883fa

File tree

3 files changed

+17
-10
lines changed

3 files changed

+17
-10
lines changed

CMakeLists.txt

Lines changed: 15 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,10 @@
44

55
cmake_minimum_required(VERSION 3.10)
66

7+
if(DEFINED PROJECT_NAME)
8+
set(PROXSUITE_AS_SUBPROJECT)
9+
endif()
10+
711
set(PROJECT_NAME proxsuite)
812
set(PROJECT_DESCRIPTION "The Advanced Proximal Optimization Toolbox")
913
set(PROJECT_URL "http://github.com/Simple-Robotics/proxsuite")
@@ -49,7 +53,6 @@ include(CMakeDependentOption)
4953
apply_default_apple_configuration()
5054

5155
option(BUILD_PYTHON_INTERFACE "Build the Python bindings" OFF)
52-
option(INSTALL_DOCUMENTATION "Generate and install the C++ documentation" OFF)
5356
option(INITIALIZE_EIGEN_WITH_NAN "Initialize Eigen objects with NAN values" OFF)
5457
option(CHECK_RUNTIME_MALLOC
5558
"Check if some memory allocations are performed at runtime" OFF)
@@ -118,6 +121,8 @@ target_link_libraries(
118121
target_include_directories(
119122
proxsuite INTERFACE "$<BUILD_INTERFACE:${CMAKE_CURRENT_LIST_DIR}/include>"
120123
"$<INSTALL_INTERFACE:${CMAKE_INSTALL_INCLUDEDIR}>")
124+
target_include_directories(
125+
proxsuite INTERFACE "$<BUILD_INTERFACE:${PROJECT_BINARY_DIR}/include>")
121126
set(EXPORTED_TARGETS_LIST proxsuite)
122127

123128
add_header_group(${PROJECT_NAME}_HEADERS)
@@ -136,15 +141,17 @@ if(BUILD_WITH_VECTORIZATION_SUPPORT)
136141
list(APPEND EXPORTED_TARGETS_LIST proxsuite-vectorized)
137142
endif()
138143

139-
install(
140-
TARGETS ${EXPORTED_TARGETS_LIST}
141-
EXPORT ${TARGETS_EXPORT_NAME}
142-
LIBRARY DESTINATION ${CMAKE_INSTALL_FULL_LIBDIR}
143-
ARCHIVE DESTINATION ${CMAKE_INSTALL_FULL_LIBDIR}
144-
RUNTIME DESTINATION ${CMAKE_INSTALL_FULL_BINDIR})
144+
if(NOT PROXSUITE_AS_SUBPROJECT)
145+
install(
146+
TARGETS ${EXPORTED_TARGETS_LIST}
147+
EXPORT ${TARGETS_EXPORT_NAME}
148+
LIBRARY DESTINATION ${CMAKE_INSTALL_FULL_LIBDIR}
149+
ARCHIVE DESTINATION ${CMAKE_INSTALL_FULL_LIBDIR}
150+
RUNTIME DESTINATION ${CMAKE_INSTALL_FULL_BINDIR})
151+
endif(NOT PROXSUITE_AS_SUBPROJECT)
145152

146153
add_subdirectory(bindings)
147-
if(BUILD_TESTING)
154+
if(BUILD_TESTING AND NOT PROXSUITE_AS_SUBPROJECT)
148155
add_subdirectory(test)
149156
add_subdirectory(examples)
150157
endif()

test/packaging/src/run-proxqp.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -18,7 +18,7 @@ main()
1818
dim, n_eq, n_in, sparsity_factor, strong_convexity_factor);
1919

2020
dense::QP<T> Qp(dim, n_eq, n_in); // create the QP object
21-
Qp.init(qp.H, qp.g, qp.A, qp.b, qp.C, qp.u, qp.l); // initialize the model
21+
Qp.init(qp.H, qp.g, qp.A, qp.b, qp.C, qp.l, qp.u); // initialize the model
2222
Qp.solve(); // solve the problem without warm start
2323

2424
auto x_wm = utils::rand::vector_rand<T>(dim);

0 commit comments

Comments
 (0)