diff options
author | Justin M. Keyes <justinkz@gmail.com> | 2018-09-24 23:18:48 +0200 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-09-24 23:18:48 +0200 |
commit | 200c8b9ccaa32b81ffa1aa4b691d763e8f67b7fa (patch) | |
tree | 89bac89d15ba0e51125b8ed178fa195b42103d77 | |
parent | 044cd242e734cb23656c66e78d9f1f4b11ed2383 (diff) | |
download | rneovim-200c8b9ccaa32b81ffa1aa4b691d763e8f67b7fa.tar.gz rneovim-200c8b9ccaa32b81ffa1aa4b691d763e8f67b7fa.tar.bz2 rneovim-200c8b9ccaa32b81ffa1aa4b691d763e8f67b7fa.zip |
build: Unify USE_BUNDLED, USE_BUNDLED_DEPS (#9046)
It's confusing that the Makefile uses a different name than CMake.
-rw-r--r-- | Makefile | 11 | ||||
-rw-r--r-- | README.md | 2 |
2 files changed, 7 insertions, 6 deletions
@@ -44,10 +44,11 @@ BUILD_CMD = $(BUILD_TOOL) $(VERBOSE_FLAG) # Extra CMake flags which extend the default set CMAKE_EXTRA_FLAGS ?= DEPS_CMAKE_FLAGS ?= -USE_BUNDLED_DEPS ?= +# Back-compat: USE_BUNDLED_DEPS was the old name. +USE_BUNDLED ?= $(USE_BUNDLED_DEPS) -ifneq (,$(USE_BUNDLED_DEPS)) - BUNDLED_CMAKE_FLAG := -DUSE_BUNDLED=$(USE_BUNDLED_DEPS) +ifneq (,$(USE_BUNDLED)) + BUNDLED_CMAKE_FLAG := -DUSE_BUNDLED=$(USE_BUNDLED) endif ifneq (,$(findstring functionaltest-lua,$(MAKECMDGOALS))) @@ -76,12 +77,12 @@ build/.ran-cmake: | deps touch $@ deps: | build/.ran-third-party-cmake -ifeq ($(call filter-true,$(USE_BUNDLED_DEPS)),) +ifeq ($(call filter-true,$(USE_BUNDLED)),) +$(BUILD_CMD) -C $(DEPS_BUILD_DIR) endif build/.ran-third-party-cmake: -ifeq ($(call filter-true,$(USE_BUNDLED_DEPS)),) +ifeq ($(call filter-true,$(USE_BUNDLED)),) mkdir -p $(DEPS_BUILD_DIR) cd $(DEPS_BUILD_DIR) && \ $(CMAKE_PRG) -G '$(BUILD_TYPE)' $(BUNDLED_CMAKE_FLAG) $(BUNDLED_LUA_CMAKE_FLAG) \ @@ -59,7 +59,7 @@ To list all targets: cmake --build build --target help -To skip "bundled" dependencies define `USE_BUNDLED_DEPS=NO` (CMake option: `USE_BUNDLED=NO`). +To skip "bundled" (`third-party/*`) dependencies define `USE_BUNDLED=NO`. See [the wiki](https://github.com/neovim/neovim/wiki/Building-Neovim) for details. |