Skip to content

Commit

Permalink
try escape the list?
Browse files Browse the repository at this point in the history
  • Loading branch information
TingDaoK committed Oct 11, 2024
1 parent d62a7bf commit 0fa285e
Showing 1 changed file with 2 additions and 7 deletions.
9 changes: 2 additions & 7 deletions cmake/AwsPrebuildDependency.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -26,17 +26,14 @@ function(aws_prebuild_dependency)
file(MAKE_DIRECTORY ${depBinaryDir})
message(STATUS "XXXXXXXXXXXX depBinaryDir ${depBinaryDir}")
message(STATUS "XXXXXXXXXXXX CMAKE_COMMAND ${CMAKE_COMMAND}")
if(WIN32)
list(JOIN CMAKE_PREFIX_PATH ";" PREFIX_PATH_STR)
else()
list(JOIN CMAKE_PREFIX_PATH ":" PREFIX_PATH_STR)
endif()
list(JOIN CMAKE_PREFIX_PATH "\\;" PREFIX_PATH_STR)
message(STATUS "XXXXXXXXXXXX PREFIX_PATH_STR ${PREFIX_PATH_STR}")

# For execute_process to accept a dynamically constructed command, it should be passed in a list format.
set(cmakeCommand "${CMAKE_COMMAND}")
list(APPEND cmakeCommand -S ${AWS_PREBUILD_SOURCE_DIR})
list(APPEND cmakeCommand -DCMAKE_BUILD_TYPE=${CMAKE_BUILD_TYPE})
list(APPEND cmakeCommand -DCMAKE_PREFIX_PATH=${PREFIX_PATH_STR})
list(APPEND cmakeCommand -DCMAKE_INSTALL_PREFIX=${depInstallDir})
list(APPEND cmakeCommand -DCMAKE_INSTALL_RPATH=${CMAKE_INSTALL_RPATH})
list(APPEND cmakeCommand -DBUILD_SHARED_LIBS=${BUILD_SHARED_LIBS})
Expand All @@ -47,8 +44,6 @@ function(aws_prebuild_dependency)
endif()

message(STATUS "XXXXXXXXXXXX cmakeCommand ${cmakeCommand}")
# Configure dependency project.
set(ENV{CMAKE_PREFIX_PATH} ${PREFIX_PATH_STR})
execute_process(
COMMAND ${cmakeCommand}
WORKING_DIRECTORY ${depBinaryDir}
Expand Down

0 comments on commit 0fa285e

Please sign in to comment.