From b15283bd27d6ac5eb784efc52ab13bc40b35a625 Mon Sep 17 00:00:00 2001
From: Gil <mcg@google.com>
Date: Sat, 13 Jan 2018 15:21:01 -0800
Subject: Build gRPC for Firestore C++ (#652)

* Clean up quoting and other minor issues

* Reorganize CMake build output

Make it clearer which parts of the output pertain to external projects.

* Use a consistent ordering of ExternalProject arguments

* Prevent the top-level build from running in parallel

This prevents spurious failures when running make -j.

* Actually parse arguments in the xcodebuild function

* Use ExternalProject features when available

  * submodule limits from CMake 3.0
  * shallow clones from CMake 3.6
  * git progress output from CMake 3.8

* Only build the parts of leveldb we need

Skip building the tools and other libraries

* Avoid installing ExternalProjects

Consume build output directly so that we can build just the targets we
need. Installing causes all targets to be built.

This doesn't matter as much for these targets but the gRPC build
includes a ton of stuff we don't need so it's worth adopting this as a
general strategy.

* Define an external build for grpc

* Test that grpc can link successfully.

* Add a FindGRPC CMake module

* Actually comment ExternalProjext_GitSource
---
 Firestore/CMakeLists.txt | 11 ++++++-----
 1 file changed, 6 insertions(+), 5 deletions(-)

(limited to 'Firestore/CMakeLists.txt')

diff --git a/Firestore/CMakeLists.txt b/Firestore/CMakeLists.txt
index 9b90815..4e009e0 100644
--- a/Firestore/CMakeLists.txt
+++ b/Firestore/CMakeLists.txt
@@ -13,9 +13,9 @@
 # limitations under the License.
 
 cmake_minimum_required(VERSION 2.8.11)
-project(firestore)
+project(firestore C CXX)
 
-set(FIREBASE_SOURCE_DIR "${CMAKE_CURRENT_LIST_DIR}/..")
+set(FIREBASE_SOURCE_DIR ${CMAKE_CURRENT_LIST_DIR}/..)
 
 # CMAKE_INSTALL_PREFIX should be passed in to this build so that it can find
 # outputs of the superbuild. This is handled automatically if run via the
@@ -23,13 +23,14 @@ set(FIREBASE_SOURCE_DIR "${CMAKE_CURRENT_LIST_DIR}/..")
 #
 # If you want to use this project directly in e.g. CLion, make sure you
 # configure this.
-set(FIREBASE_INSTALL_DIR "${CMAKE_INSTALL_PREFIX}")
+set(FIREBASE_INSTALL_DIR ${CMAKE_INSTALL_PREFIX})
 
 list(INSERT CMAKE_MODULE_PATH 0 ${FIREBASE_SOURCE_DIR}/cmake)
 include(utils)
 
 find_package(GTest REQUIRED)
 find_package(LevelDB REQUIRED)
+find_package(GRPC REQUIRED)
 
 if(APPLE)
   find_package(FirebaseCore REQUIRED)
@@ -41,7 +42,7 @@ set(CMAKE_CXX_STANDARD_REQUIRED ON)
 set(CMAKE_CXX_EXTENSIONS OFF)
 
 # Fully qualified imports, project wide
-include_directories("${FIREBASE_SOURCE_DIR}")
+include_directories(${FIREBASE_SOURCE_DIR})
 
 if(APPLE)
   # CMake has no special support for Objective-C as a distinct language but enabling modules and
@@ -57,5 +58,5 @@ if(APPLE)
 endif(APPLE)
 
 enable_testing()
-add_subdirectory(third_party/abseil-cpp EXCLUDE_FROM_ALL)
+add_subdirectory(third_party/abseil-cpp)
 add_subdirectory(core)
-- 
cgit v1.2.3