Skip to content

Commit

Permalink
Merge pull request #7967 from cloudhan/win_build_chainerx
Browse files Browse the repository at this point in the history
Fix Windows build of ChainerX
  • Loading branch information
mergify[bot] committed Aug 19, 2019
2 parents 41a0724 + e0e7856 commit 2e3ed4a
Show file tree
Hide file tree
Showing 5 changed files with 14 additions and 2 deletions.
2 changes: 1 addition & 1 deletion chainerx_cc/chainerx/native/CMakeLists.txt
Expand Up @@ -40,7 +40,7 @@ add_library(chainerx_native STATIC
if(${BLAS_FOUND})
if(DEFINED ENV{CHAINERX_BLAS_INCLUDE_DIRS})
set(BLAS_INCLUDE_DIRS $ENV{CHAINERX_BLAS_INCLUDE_DIRS})
else()
elseif(UNIX)
# Workaround for UNIX environments to guess the BLAS include directory from its library path (.so).
# This workaround is required since CMake does not set the BLAS_INCLUDE_DIRS variable.
list(GET BLAS_LIBRARIES 0 BLAS_LIBRARY)
Expand Down
3 changes: 3 additions & 0 deletions chainerx_cc/third_party/abseil.cmake
Expand Up @@ -13,4 +13,7 @@ ExternalProject_Add(pybind11
BUILD_COMMAND ""
INSTALL_COMMAND ""
TEST_COMMAND ""
CMAKE_ARGS
-DCMAKE_INSTALL_MESSAGE=LAZY
-DCMAKE_POSITION_INDEPENDENT_CODE=ON
)
5 changes: 4 additions & 1 deletion chainerx_cc/third_party/gsl-lite.cmake
Expand Up @@ -11,4 +11,7 @@ ExternalProject_Add(gsl-lite
BUILD_COMMAND ""
INSTALL_COMMAND ""
TEST_COMMAND ""
)
CMAKE_ARGS
-DCMAKE_INSTALL_MESSAGE=LAZY
-DCMAKE_POSITION_INDEPENDENT_CODE=ON
)
3 changes: 3 additions & 0 deletions chainerx_cc/third_party/gtest.cmake
Expand Up @@ -12,4 +12,7 @@ ExternalProject_Add(googletest
BUILD_COMMAND ""
INSTALL_COMMAND ""
TEST_COMMAND ""
CMAKE_ARGS
-DCMAKE_INSTALL_MESSAGE=LAZY
-DCMAKE_POSITION_INDEPENDENT_CODE=ON
)
3 changes: 3 additions & 0 deletions chainerx_cc/third_party/pybind11.cmake
Expand Up @@ -11,4 +11,7 @@ ExternalProject_Add(pybind11
BUILD_COMMAND ""
INSTALL_COMMAND ""
TEST_COMMAND ""
CMAKE_ARGS
-DCMAKE_INSTALL_MESSAGE=LAZY
-DCMAKE_POSITION_INDEPENDENT_CODE=ON
)

0 comments on commit 2e3ed4a

Please sign in to comment.