Merge branch 'dev' of https://github.com/firebase/firebase-cpp-sdk into dev

Author: jonsimantov

.github/workflows/android.yml

@@ -31,8 +31,8 @@ jobs:
 
       - name: Set env variables for subsequent steps (all)
         run: |
-          echo "::set-env name=MATRIX_UNIQUE_NAME::${{ matrix.os }}-${{ matrix.architecture }}"
-          echo "::set-env name=GHA_INSTALL_CCACHE::1"
+          echo "MATRIX_UNIQUE_NAME=${{ matrix.os }}-${{ matrix.architecture }}" >> $GITHUB_ENV
+          echo "GHA_INSTALL_CCACHE=1" >> $GITHUB_ENV
 
       - name: Setup python
         uses: actions/setup-python@v2

.github/workflows/cpp-packaging.yml

@@ -229,9 +229,9 @@ jobs:
 
       - name: Set env variables for subsequent steps (all)
         run: |
-          echo "::set-env name=VCPKG_RESPONSE_FILE::external/vcpkg_${{ matrix.vcpkg_triplet }}_response_file.txt"
-          echo "::set-env name=MATRIX_UNIQUE_NAME::${{ matrix.os }}-${{ matrix.build_type }}-${{ matrix.architecture }}-${{ matrix.python_version }}"
-          echo "::set-env name=SDK_NAME::${{ matrix.sdk_platform }}-${{ matrix.architecture }}-${{ matrix.build_type }}-${{ matrix.linkage }}"
+          echo "VCPKG_RESPONSE_FILE=external/vcpkg_${{ matrix.vcpkg_triplet }}_response_file.txt" >> $GITHUB_ENV
+          echo "MATRIX_UNIQUE_NAME=${{ matrix.os }}-${{ matrix.build_type }}-${{ matrix.architecture }}-${{ matrix.python_version }}" >> $GITHUB_ENV
+          echo "SDK_NAME=${{ matrix.sdk_platform }}-${{ matrix.architecture }}-${{ matrix.build_type }}-${{ matrix.linkage }}" >> $GITHUB_ENV
 
       - name: Add msbuild to PATH (windows)
         if: startsWith(matrix.os, 'windows')

.github/workflows/desktop.yml

@@ -1,6 +1,6 @@
 name: Desktop Builds
 
-on: 
+on:
   pull_request:
     types: [opened, reopened, synchronize]
 
@@ -9,26 +9,34 @@ env:
 
 jobs:
   build:
-    name: ${{ matrix.os }}-${{ matrix.build_type }}-${{ matrix.architecture }}-${{ matrix.python_version }}
+    name: ${{ matrix.os }}-${{ matrix.build_type }}-${{ matrix.architecture }}-${{ matrix.msvc_runtime}}
     runs-on: ${{ matrix.os }}
     strategy:
       fail-fast: false
       matrix:
         os: [windows-latest, ubuntu-latest, macos-latest]
         build_type: ["Release", "Debug"]
-        architecture: ["x64",]
+        architecture: ["x64", "x86"]
+        msvc_runtime: ["static", "dynamic"]
         python_version: [3.7]
         include:
         - os: windows-latest
-          architecture: "x64"
-          vcpkg_triplet: "x64-windows-static"
-          msbuild_platform: "x64"
+          vcpkg_triplet_suffix: "windows-static"
+        - os: windows-latest
+          msvc_runtime: "dynamic"
+          vcpkg_triplet_suffix: "windows-static-md"
         - os: ubuntu-latest
-          architecture: "x64"
-          vcpkg_triplet: "x64-linux"
+          vcpkg_triplet_suffix: "linux"
+        - os: macos-latest
+          vcpkg_triplet_suffix: "osx"
+
+        exclude:
         - os: macos-latest
-          architecture: "x64"
-          vcpkg_triplet: "x64-osx"
+          architecture: "x86"
+        - os: macos-latest
+          msvc_runtime: "dynamic"
+        - os: ubuntu-latest
+          msvc_runtime: "dynamic"
 
     steps:
       - uses: actions/checkout@v2
@@ -37,15 +45,15 @@ jobs:
 
       - name: Set env variables for subsequent steps (all)
         run: |
-          echo "::set-env name=VCPKG_RESPONSE_FILE::external/vcpkg_${{ matrix.vcpkg_triplet }}_response_file.txt"
-          echo "::set-env name=MATRIX_UNIQUE_NAME::${{ matrix.os }}-${{ matrix.build_type }}-${{ matrix.architecture }}-${{ matrix.python_version }}"
+          echo "VCPKG_RESPONSE_FILE=external/vcpkg_custom_data/response_files/${{ matrix.architecture }}-${{ matrix.vcpkg_triplet_suffix }}.txt" >> $GITHUB_ENV
+          echo "MATRIX_UNIQUE_NAME=${{ matrix.os }}-${{ matrix.build_type }}-${{ matrix.architecture }}-${{ matrix.msvc_runtime }}" >> $GITHUB_ENV
 
       - name: Cache vcpkg C++ dependencies
         id: cache_vcpkg
         uses: actions/cache@v2
         with:
           path: external/vcpkg/installed
-          key: dev-vcpkg-${{ matrix.vcpkg_triplet }}-${{ hashFiles(format('{0}', env.VCPKG_RESPONSE_FILE)) }}-${{ hashFiles('.git/modules/external/vcpkg/HEAD') }}
+          key: dev-vcpkg-${{ matrix.architecture }}-${{ matrix.vcpkg_triplet_suffix }}-${{ matrix.msvc_runtime }}-${{ hashFiles(format('{0}', env.VCPKG_RESPONSE_FILE)) }}-${{ hashFiles('.git/modules/external/vcpkg/HEAD') }}
 
       - name: Cache ccache files
         if: startsWith(matrix.os, 'ubuntu') || startsWith(matrix.os, 'macos')
@@ -59,15 +67,15 @@ jobs:
         uses: actions/setup-python@v2
         with:
           python-version: ${{ matrix.python_version }}
-          architecture: ${{ matrix.architecture }}
+          architecture: 'x64'
 
       - name: Install prerequisites
         run: |
           python scripts/gha/install_prereqs_desktop.py
 
       - name: Build SDK
         run: |
-          python scripts/gha/build_desktop.py --build_tests --arch "${{ matrix.architecture }}" --config "${{ matrix.build_type }}"
+          python scripts/gha/build_desktop.py --build_tests --arch "${{ matrix.architecture }}" --config "${{ matrix.build_type }}" --msvc_runtime_library "${{ matrix.msvc_runtime }}"
 
       - name: Stats for ccache (mac and linux)
         if: startsWith(matrix.os, 'ubuntu') || startsWith(matrix.os, 'macos')
@@ -82,23 +90,30 @@ jobs:
           find build -name "*.a"
           find build -name "*.so"
 
+      - name: Inspect firebase libraries for cpu arch and msvc runtime.
+        shell: bash
+        run: |
+          python scripts/gha/inspect_built_libraries.py build/
+        continue-on-error: true
+
       - name: Prep crash support (linux)
-        # Ensures directory structure is in place to capture core files on Linux. 
+        # Ensures directory structure is in place to capture core files on Linux.
         if: startsWith(matrix.build_type, 'Debug') && startsWith(matrix.os, 'ubuntu')
         shell: bash
         run: |
           echo '/tmp/cores/core.%E' | sudo tee /proc/sys/kernel/core_pattern
           sudo mkdir -p /tmp/cores/
           sudo rm -rf /tmp/cores/*
-          
+
       - name: Prep crash support (mac)
         # Ensures no lingering crashes from the previous run.
         if: startsWith(matrix.build_type, 'Debug') && startsWith(matrix.os, 'macos')
         run: |
           rm -rf ~/Library/Logs/DiagnosticReports/*
-          
+
       - name: Run unit tests (windows & macos)
-        if: startsWith(matrix.os, 'windows') || startsWith(matrix.os, 'macos')
+        # TODO: Enable tests for x86 once they are all working
+        if: (startsWith(matrix.os, 'windows') || startsWith(matrix.os, 'macos')) && matrix.architecture != 'x86'
         env:
           LANG: en_US
         run: |
@@ -110,14 +125,15 @@ jobs:
         # platform-specific `ulimit` to enable crash collection.  The ulimit
         # command must be invoked in same shell instance of that runs the
         # tests.
-        if: startsWith(matrix.os, 'ubuntu')
+        # TODO: Enable tests for x86 once they are all working
+        if: startsWith(matrix.os, 'ubuntu') && matrix.architecture != 'x86'
         env:
           LANG: en_US
         run: |
-          ulimit -c unlimited 
+          ulimit -c unlimited
           cd build
           sudo ctest --verbose
-      
+
       - name: Prep bins for achive (linux)
         # Copies all of the binary files into one directory for ease in
         # archiving. Directory contents are then marked readable for the
@@ -147,7 +163,7 @@ jobs:
         shell: bash
         if: failure() && startsWith(matrix.build_type, 'Debug') && startsWith(matrix.os, 'macos')
         run: |
-          sudo chmod -R +rwx ~/Library/Logs/DiagnosticReports/* || :           
+          sudo chmod -R +rwx ~/Library/Logs/DiagnosticReports/* || :
 
       - name: Archive crashes and binaries (linux)
         uses: actions/upload-artifact@v2
@@ -166,4 +182,3 @@ jobs:
           path: |
             ~/Library/Logs/DiagnosticReports/*
             /tmp/bins/*
-

.github/workflows/integration_tests.yml

@@ -66,20 +66,20 @@ jobs:
 
       - name: Set env vars (ubuntu)
         if: startsWith(matrix.os, 'ubuntu')
-        run: echo "::set-env name=VCPKG_TRIPLET::x64-linux"
+        run: echo "VCPKG_TRIPLET=x64-linux" >> $GITHUB_ENV
       - name: Set env vars (macos)
         if: startsWith(matrix.os, 'macos')
-        run: echo "::set-env name=VCPKG_TRIPLET::x64-osx"
+        run: echo "VCPKG_TRIPLET=x64-osx" >> $GITHUB_ENV
       - name: Set env vars (windows)
         if: startsWith(matrix.os, 'windows')
-        run: echo "::set-env name=VCPKG_TRIPLET::x64-windows-static"
+        run: echo "VCPKG_TRIPLET=x64-windows-static" >> $GITHUB_ENV
       - name: Set env vars(all)
-        run: echo "::set-env name=VCPKG_RESPONSE_FILE::external/vcpkg_${{ env.VCPKG_TRIPLET }}_response_file.txt"
+        run: echo "VCPKG_RESPONSE_FILE=external/vcpkg_${{ env.VCPKG_TRIPLET }}_response_file.txt" >> $GITHUB_ENV
 
       - name: Add msbuild to PATH (windows)
         if: startsWith(matrix.os, 'windows')
         uses: microsoft/[email protected]
-  
+
       - name: Cache vcpkg C++ dependencies
         if: matrix.target_platform == 'Desktop'
         id: cache_vcpkg
@@ -97,7 +97,7 @@ jobs:
         if: matrix.target_platform == 'Desktop'
         run: |
           python scripts/gha/install_prereqs_desktop.py
-          
+
       - name: Install SDK Android prerequisites
         if: matrix.target_platform == 'Android'
         shell: bash
@@ -108,27 +108,27 @@ jobs:
         if: matrix.target_platform == 'iOS'
         run: |
           build_scripts/ios/install_prereqs.sh
-          
+
       - name: Prepare for integration tests
         run: |
           pip install -r scripts/gha/requirements.txt
           python scripts/gha/restore_secrets.py --passphrase "${{ secrets.TEST_SECRET }}"
-          
+
       - name: Build integration tests (and run Desktop tests)
         # The set up script for Android will download the NDK here.
         env:
           NDK_ROOT: '/tmp/android-ndk-r16b'
         run: |
           python scripts/gha/build_testapps.py --t ${{ github.event.inputs.apis }} --p ${{ matrix.target_platform }} --output_directory ${{ github.workspace }} --use_vcpkg --execute_desktop_testapp --noadd_timestamp
-      
+
       # Workaround for https://github.com/GoogleCloudPlatform/github-actions/issues/100
       # Must be run after the Python setup action
       - name: Set CLOUDSDK_PYTHON (Windows)
         if: startsWith(matrix.os, 'windows') && !cancelled()
-        run: echo "::set-env name=CLOUDSDK_PYTHON::${{env.pythonLocation}}\python.exe"
+        run: echo "CLOUDSDK_PYTHON=${{env.pythonLocation}}\python.exe" >> $GITHUB_ENV
       - name: Install Cloud SDK
         if: ${{ !cancelled() }}
-        uses: GoogleCloudPlatform/github-actions/setup-gcloud@master 
+        uses: GoogleCloudPlatform/github-actions/setup-gcloud@master
       - name: Upload Desktop Artifacts to GCS
         if: matrix.target_platform == 'Desktop' && !cancelled()
         run: |

.github/workflows/ios.yml

@@ -1,6 +1,6 @@
 name: iOS Builds
 
-on: 
+on:
   pull_request:
     types: [opened, reopened, synchronize]
 

CMakeLists.txt

@@ -83,13 +83,12 @@ project (firebase NONE)
 enable_language(C)
 enable_language(CXX)
 
-set(compiler_cache_program ccache)
-
-find_program(compilercache_program ${compiler_cache_program})
-if(compilercache_program)
-  set_property(GLOBAL PROPERTY RULE_LAUNCH_COMPILE "${compiler_cache_program}")
-  set_property(GLOBAL PROPERTY RULE_LAUNCH_LINK "${compiler_cache_program}")
-  message( STATUS "Found compiler cache program : ${compiler_cache_program}" )
+if(NOT DEFINED CMAKE_CXX_COMPILER_LAUNCHER)
+  find_program(CCACHE_PROGRAM ccache)
+  if (CCACHE_PROGRAM)
+    set(CMAKE_C_COMPILER_LAUNCHER "${CCACHE_PROGRAM}" CACHE STRING "Compiler launcher for C")
+    set(CMAKE_CXX_COMPILER_LAUNCHER "${CCACHE_PROGRAM}" CACHE STRING "Compiler launcher for CXX")
+  endif()
 endif()
 
 list(INSERT CMAKE_MODULE_PATH 0 ${CMAKE_CURRENT_LIST_DIR}/cmake)
@@ -122,23 +121,20 @@ set(FIREBASE_GEN_FILE_DIR ${CMAKE_BINARY_DIR}/generated)
 # Directory for any shared scripts.
 set(FIREBASE_SCRIPT_DIR ${CMAKE_CURRENT_LIST_DIR})
 
-if (MSVC AND MSVC_RUNTIME_LIBRARY_STATIC)
-  # Use Multithreaded static runtime libraries for MSVC 
-  # if MSVC_RUNTIME_LIBRARY_STATIC is set.
-  # Default value if nothing is specified is /MD (dll dynamic runtime).
-  set(CMAKE_C_FLAGS_DEBUG "${CMAKE_C_FLAGS_DEBUG} /MTd")
-  set(CMAKE_C_FLAGS_RELEASE "${CMAKE_C_FLAGS_RELEASE} /MT")
-  set(CMAKE_CXX_FLAGS_DEBUG "${CMAKE_CXX_FLAGS_DEBUG} /MTd")
-  set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE} /MT")
-endif()
-  
-if (FIREBASE_CPP_BUILD_TESTS AND MSVC)
-  # GoogleTest requires MSVC to compile with the static version of the runtime
-  # library, so define the appropriate runtime flag, before adding libraries.
-  set(CMAKE_C_FLAGS_DEBUG "${CMAKE_C_FLAGS_DEBUG} /MTd")
-  set(CMAKE_C_FLAGS_RELEASE "${CMAKE_C_FLAGS_RELEASE} /MT")
-  set(CMAKE_CXX_FLAGS_DEBUG "${CMAKE_CXX_FLAGS_DEBUG} /MTd")
-  set(CMAKE_CXX_FLAGS_RELEASE "${CMAKE_CXX_FLAGS_RELEASE} /MT")
+if (MSVC)
+  if (MSVC_RUNTIME_LIBRARY_STATIC)
+    add_compile_options(
+      $<$<CONFIG:>:/MT>
+      $<$<CONFIG:Debug>:/MTd>
+      $<$<CONFIG:Release>:/MT>
+  )
+  else()
+    add_compile_options(
+      $<$<CONFIG:>:/MD>
+      $<$<CONFIG:Debug>:/MDd>
+      $<$<CONFIG:Release>:/MD>
+  )
+  endif()
 endif()
 
 # Set variables that are used by download_external_sources.

build_scripts/android/install_prereqs.sh

@@ -4,13 +4,13 @@ if [[ $(uname) == "Darwin" ]]; then
     platform=darwin
     if [[ ! -z "${GHA_INSTALL_CCACHE}" ]]; then
         brew install ccache
-        echo "::set-env name=CCACHE_INSTALLED::1"
+        echo "CCACHE_INSTALLED=1" >> $GITHUB_ENV
     fi
 elif [[ $(uname) == "Linux" ]]; then
     platform=linux
     if [[ ! -z "${GHA_INSTALL_CCACHE}" ]]; then
         sudo apt install ccache
-        echo "::set-env name=CCACHE_INSTALLED::1"
+        echo "CCACHE_INSTALLED=1" >> $GITHUB_ENV
     fi
 else
     platform=windows

build_scripts/desktop/get_variant.sh

@@ -29,7 +29,7 @@ os=
 arch=
 arch_win=
 debugmode=Release
-msvc_crt_linkage=MD
+msvc_runtime_library=MD
 vs=VS2015
 stl=c++
 
@@ -99,16 +99,16 @@ for c in $(echo "${filename}" | tr "_.-" "\n\n\n"); do
 	;;
 	# Windows OS framework linkages
 	md)
-	    msvc_crt_linkage=MD
+	    msvc_runtime_library=MD
 	;;
 	dynamic)
-	    msvc_crt_linkage=MD
+	    msvc_runtime_library=MD
 	;;
 	mt)
-	    msvc_crt_linkage=MT
+	    msvc_runtime_library=MT
 	;;
 	static)
-	    msvc_crt_linkage=MT
+	    msvc_runtime_library=MT
 	    ;;
 	# Debug/Release compilation mode
 	release)
@@ -161,7 +161,7 @@ case ${os} in
 	    echo "Invalid architecture for Windows: ${arch}" 1>&2
 	    exit 1
 	fi
-	echo -n "${vs}/${msvc_crt_linkage}/${arch_win}/${debugmode}"
+	echo -n "${vs}/${msvc_runtime_library}/${arch_win}/${debugmode}"
     ;;
     ios)
 	if [[ -z "${arch}" ]]; then

cmake/toolchains/linux_32.cmake

@@ -1,3 +1,17 @@
+# Copyright 2020 Google
+#
+# Licensed under the Apache License, Version 2.0 (the "License");
+# you may not use this file except in compliance with the License.
+# You may obtain a copy of the License at
+#
+#      http://www.apache.org/licenses/LICENSE-2.0
+#
+# Unless required by applicable law or agreed to in writing, software
+# distributed under the License is distributed on an "AS IS" BASIS,
+# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+# See the License for the specific language governing permissions and
+# limitations under the License.
+
 # Toolchain file for building 32-bit Linux libraries
 
 set(CMAKE_C_FLAGS "${CMAKE_C_FLAGS} -m32")