From 0857a4ec20db4b038fb5346ed09e3253842f0169 Mon Sep 17 00:00:00 2001 From: Don Hinton Date: Thu, 23 May 2019 15:03:22 +0000 Subject: [PATCH] [cmake] When getting Ninja version, don't include CMakeNinjaFindMake which doesn't play well with passing CMAKE_MAKE_PROGRAM from the commandline without a path. MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Fixes a bug introduced in r361280. Thanks to Mikael Holmén for reporting this! llvm-svn: 361501 --- llvm/cmake/config-ix.cmake | 13 +++++-------- 1 file changed, 5 insertions(+), 8 deletions(-) diff --git a/llvm/cmake/config-ix.cmake b/llvm/cmake/config-ix.cmake index 36ec98a93862..e1e5cd3198b2 100644 --- a/llvm/cmake/config-ix.cmake +++ b/llvm/cmake/config-ix.cmake @@ -555,14 +555,11 @@ set(LLVM_BINUTILS_INCDIR "" CACHE PATH "PATH to binutils/include containing plugin-api.h for gold plugin.") if(CMAKE_GENERATOR STREQUAL "Ninja") - include(CMakeNinjaFindMake OPTIONAL) - if(CMAKE_MAKE_PROGRAM) - execute_process(COMMAND ${CMAKE_MAKE_PROGRAM} --version - OUTPUT_VARIABLE NINJA_VERSION - OUTPUT_STRIP_TRAILING_WHITESPACE) - set(NINJA_VERSION ${NINJA_VERSION} CACHE STRING "Ninja version number" FORCE) - message(STATUS "Ninja version: ${NINJA_VERSION}") - endif() + execute_process(COMMAND ${CMAKE_MAKE_PROGRAM} --version + OUTPUT_VARIABLE NINJA_VERSION + OUTPUT_STRIP_TRAILING_WHITESPACE) + set(NINJA_VERSION ${NINJA_VERSION} CACHE STRING "Ninja version number" FORCE) + message(STATUS "Ninja version: ${NINJA_VERSION}") endif() if(CMAKE_GENERATOR STREQUAL "Ninja" AND