about summary refs log tree commit diff
diff options
context:
space:
mode:
authorbors <bors@rust-lang.org>2016-12-06 03:45:49 +0000
committerbors <bors@rust-lang.org>2016-12-06 03:45:49 +0000
commitf7c93c07b8533e1d38395cc2d9d37cd2d9bec978 (patch)
tree2f8e8ebe5bcca25a01eb696268da149a6967121c
parent09991241fd75ec69281d06c8f907d2e9f61ac5e2 (diff)
parentf83eb4009e3e5a52bcb2dddc9fa7764aa9c12841 (diff)
downloadrust-f7c93c07b8533e1d38395cc2d9d37cd2d9bec978.tar.gz
rust-f7c93c07b8533e1d38395cc2d9d37cd2d9bec978.zip
Auto merge of #38128 - cardoe:req-cmake-only-for-llvm, r=alexcrichton
configure: only req CMake if we're building LLVM

CMake is only necessary if LLVM is going to be built and not in any
other case.

Signed-off-by: Doug Goldstein <cardoe@cardoe.com>
-rwxr-xr-xconfigure9
1 files changed, 6 insertions, 3 deletions
diff --git a/configure b/configure
index 5311bf4b064..483471604cb 100755
--- a/configure
+++ b/configure
@@ -848,7 +848,10 @@ then
 fi
 
 # For building LLVM
-probe_need CFG_CMAKE cmake
+if [ -z "$CFG_LLVM_ROOT" ]
+then
+  probe_need CFG_CMAKE cmake
+fi
 
 # On MacOS X, invoking `javac` pops up a dialog if the JDK is not
 # installed. Since `javac` is only used if `antlr4` is available,
@@ -1471,7 +1474,7 @@ fi
 step_msg "configuring submodules"
 
 # Have to be in the top of src directory for this
-if [ -z $CFG_DISABLE_MANAGE_SUBMODULES ] && [ -z $CFG_ENABLE_RUSTBUILD ]
+if [ -z "$CFG_DISABLE_MANAGE_SUBMODULES" ] && [ -z "$CFG_ENABLE_RUSTBUILD" ]
 then
     cd ${CFG_SRC_DIR}
 
@@ -1547,7 +1550,7 @@ do
     then
         msg "not configuring LLVM, rustbuild in use"
         do_reconfigure=0
-    elif [ -z $CFG_LLVM_ROOT ]
+    elif [ -z "$CFG_LLVM_ROOT" ]
     then
         LLVM_BUILD_DIR=${CFG_BUILD_DIR}$t/llvm
         LLVM_INST_DIR=$LLVM_BUILD_DIR