Skip to content

Commit

Permalink
Fix setting CMAKE_GENERATOR "Ninja" in config.cmake (apache#17742)
Browse files Browse the repository at this point in the history
This is not supported anymore, as we moved from `cmake -C ../config.cmake ..`
syntax to `cmake ..`. Without `-C`, the config file is read too late and we
can't modify the generator anymore. Ask users to run `cmake -GNinja ..` instead.
  • Loading branch information
leezu authored Mar 4, 2020
1 parent 6368977 commit 64d8e0f
Show file tree
Hide file tree
Showing 5 changed files with 2 additions and 8 deletions.
2 changes: 0 additions & 2 deletions config/darwin.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -55,8 +55,6 @@ set(USE_TVM_OP OFF CACHE BOOL "Enable use of TVM operator build system.")
#---------------------
# Compilers
#--------------------
set(CMAKE_GENERATOR "Ninja" CACHE STRING "Build Tool Generator used by CMake")

# Compilers are usually autodetected. Uncomment and modify the next 3 lines to
# choose manually:

Expand Down
2 changes: 0 additions & 2 deletions config/linux.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -73,8 +73,6 @@ set(USE_TVM_OP OFF CACHE BOOL "Enable use of TVM operator build system.")
#---------------------
# Compilers
#--------------------
set(CMAKE_GENERATOR "Ninja" CACHE STRING "Build Tool Generator used by CMake")

# Compilers are usually autodetected. Uncomment and modify the next 3 lines to
# choose manually:

Expand Down
2 changes: 0 additions & 2 deletions config/linux_gpu.cmake
Original file line number Diff line number Diff line change
Expand Up @@ -73,8 +73,6 @@ set(USE_TVM_OP OFF CACHE BOOL "Enable use of TVM operator build system.")
#---------------------
# Compilers
#--------------------
set(CMAKE_GENERATOR "Ninja" CACHE STRING "Build Tool Generator used by CMake")

# Compilers are usually autodetected. Uncomment and modify the next 3 lines to
# choose manually:

Expand Down
2 changes: 1 addition & 1 deletion docs/static_site/src/pages/get_started/osx_setup.md
Original file line number Diff line number Diff line change
Expand Up @@ -110,7 +110,7 @@ the guide in [Math Library Selection](build_from_source#math-library-selection).
```bash
rm -rf build
mkdir -p build && cd build
cmake ..
cmake -GNinja ..
cmake --build .
```

Expand Down
2 changes: 1 addition & 1 deletion docs/static_site/src/pages/get_started/ubuntu_setup.md
Original file line number Diff line number Diff line change
Expand Up @@ -130,7 +130,7 @@ the guide in [Math Library Selection](build_from_source#math-library-selection).
```bash
rm -rf build
mkdir -p build && cd build
cmake ..
cmake -GNinja ..
cmake --build .
```

Expand Down

0 comments on commit 64d8e0f

Please sign in to comment.