From 49f2493e14cd68ecc0e08ad2d9fc75739e419a3b Mon Sep 17 00:00:00 2001 From: Gil Date: Wed, 11 Jul 2018 19:32:24 -0700 Subject: Convert the gRPC external build to use an HTTP download (#1520) * Add a download-only boringssl ExternalProject Match boringssl version to grpc submodule version * Split grpc into separate download and build tasks This allows other external targets to download into the gRPC source tree before grpc is actually built. * Convert grpc to an http download --- cmake/external/grpc.cmake | 45 ++++++++++++++++++++++++++++++--------------- 1 file changed, 30 insertions(+), 15 deletions(-) (limited to 'cmake/external/grpc.cmake') diff --git a/cmake/external/grpc.cmake b/cmake/external/grpc.cmake index d35eb15..310d238 100644 --- a/cmake/external/grpc.cmake +++ b/cmake/external/grpc.cmake @@ -13,7 +13,6 @@ # limitations under the License. include(ExternalProject) -include(ExternalProjectFlags) include(external/c-ares) include(external/protobuf) include(external/zlib) @@ -29,15 +28,11 @@ if(GRPC_ROOT) return() endif() -set( - GIT_SUBMODULES - third_party/boringssl -) - set( CMAKE_ARGS -DCMAKE_BUILD_TYPE:STRING=${CMAKE_BUILD_TYPE} -DBUILD_SHARED_LIBS:BOOL=OFF + -DgRPC_INSTALL:BOOL=OFF -DgRPC_BUILD_TESTS:BOOL=OFF # TODO(rsgowman): We're currently building nanopb twice; once via grpc, and @@ -105,23 +100,43 @@ if(ZLIB_FOUND) endif() -ExternalProject_GitSource( - GRPC_GIT - GIT_REPOSITORY "https://github.com/grpc/grpc.git" - GIT_TAG "v1.8.3" - GIT_SUBMODULES ${GIT_SUBMODULES} -) - ExternalProject_Add( - grpc + grpc-download DEPENDS c-ares protobuf zlib - ${GRPC_GIT} + DOWNLOAD_DIR ${FIREBASE_DOWNLOAD_DIR} + DOWNLOAD_NAME grpc-1.8.3.tar.gz + URL https://github.com/grpc/grpc/archive/v1.8.3.tar.gz + URL_HASH SHA256=c14bceddc6475a09927a815811a8161cdfa7acb445262835da6bc24da9842c92 + + PREFIX ${PROJECT_BINARY_DIR} + SOURCE_DIR ${PROJECT_BINARY_DIR}/src/grpc + + CONFIGURE_COMMAND "" + BUILD_COMMAND "" + TEST_COMMAND "" + INSTALL_COMMAND "" +) + +# gRPC depends upon these projects, so from an IWYU point of view should +# include these files. Unfortunately gRPC's build requires these to be +# subdirectories in its own source tree and CMake's ExternalProject download +# step clears the source tree so these must be declared to depend upon the grpc +# target. ExternalProject dependencies must already exist when declared so +# these must come after the ExternalProject_Add block above. +include(external/boringssl) + +ExternalProject_Add( + grpc + DEPENDS + boringssl + grpc-download PREFIX ${PROJECT_BINARY_DIR} + SOURCE_DIR ${PROJECT_BINARY_DIR}/src/grpc CMAKE_ARGS ${CMAKE_ARGS} -- cgit v1.2.3