Skip to content

Commit e45d14e

Browse files
committed
Merge branch 'dev' into bugfix/integration-tests-mac-openssl-error
2 parents c7be098 + 81f28c1 commit e45d14e

File tree

4 files changed

+29
-4
lines changed

4 files changed

+29
-4
lines changed

.github/workflows/cpp-packaging.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -717,7 +717,7 @@ jobs:
717717
run: echo "CLOUDSDK_PYTHON=${{env.pythonLocation}}\python.exe" >> $GITHUB_ENV
718718
- name: Install Cloud SDK
719719
if: ${{ !cancelled() }}
720-
uses: GoogleCloudPlatform/github-actions/setup-gcloud@master
720+
uses: google-github-actions/setup-gcloud@master
721721
- name: Upload Desktop Artifacts to GCS
722722
if: ${{ !cancelled() }}
723723
run: |

.github/workflows/integration_tests.yml

Lines changed: 8 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -128,6 +128,13 @@ jobs:
128128
pip install -r scripts/gha/requirements.txt
129129
python scripts/gha/restore_secrets.py --passphrase "${{ secrets.TEST_SECRET }}"
130130
131+
- name: Install OpenSSL (windows)
132+
if: matrix.target_platform == 'Desktop' &&
133+
matrix.ssl_variant == 'openssl' &&
134+
startsWith(matrix.os, 'windows')
135+
run: |
136+
choco install openssl -r
137+
131138
- name: Build integration tests
132139
shell: bash
133140
run: |
@@ -150,7 +157,7 @@ jobs:
150157
run: echo "CLOUDSDK_PYTHON=${{env.pythonLocation}}\python.exe" >> $GITHUB_ENV
151158
- name: Install Cloud SDK
152159
if: ${{ !cancelled() }}
153-
uses: GoogleCloudPlatform/github-actions/setup-gcloud@master
160+
uses: google-github-actions/setup-gcloud@master
154161
- name: Upload Desktop Artifacts to GCS
155162
if: matrix.target_platform == 'Desktop' && !cancelled()
156163
run: |

app/CMakeLists.txt

Lines changed: 16 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -298,6 +298,19 @@ else()
298298
"${app_desktop_HDRS}")
299299
endif()
300300

301+
if(ANDROID OR IOS)
302+
# If building for Android or iOS, include the Flatbuffers source files directly.
303+
set(app_flatbuffers_srcs
304+
${FLATBUFFERS_SOURCE_DIR}/src/idl_parser.cpp
305+
${FLATBUFFERS_SOURCE_DIR}/src/idl_gen_text.cpp
306+
${FLATBUFFERS_SOURCE_DIR}/src/reflection.cpp
307+
${FLATBUFFERS_SOURCE_DIR}/src/util.cpp)
308+
set(app_flatbuffers_lib)
309+
else()
310+
set(app_flatbuffers_srcs)
311+
set(app_flatbuffers_lib flatbuffers)
312+
endif()
313+
301314
add_library(firebase_app STATIC
302315
${log_SRCS}
303316
${log_HDRS}
@@ -311,7 +324,8 @@ add_library(firebase_app STATIC
311324
memory/atomic.h
312325
meta/move.h
313326
memory/unique_ptr.h
314-
memory/shared_ptr.h)
327+
memory/shared_ptr.h
328+
${app_flatbuffers_srcs})
315329

316330
set_property(TARGET firebase_app PROPERTY FOLDER "Firebase Cpp")
317331

@@ -339,7 +353,7 @@ target_compile_definitions(firebase_app
339353
# firebase_app has a dependency on flatbuffers, which needs to be included.
340354
target_link_libraries(firebase_app
341355
PRIVATE
342-
flatbuffers
356+
${app_flatbuffers_lib}
343357
${LIBSECRET_LIBRARIES}
344358
)
345359
# Automatically include headers that might not be declared.

build_scripts/android/package.sh

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -53,5 +53,9 @@ for product in ${product_list[*]}; do
5353
done
5454
# Copy the top-level Proguard files in as well.
5555
cp -f "${sourcepath}/${product}/build/Release/${product}.pro" "${destpath}/libs/android/"
56+
# Copy the special messaging aar file, but only if messaging was built.
57+
if [[ "${product}" == "messaging" ]]; then
58+
cp -f "${sourcepath}/messaging/messaging_java/build/outputs/aar/messaging_java-release.aar" "${destpath}/libs/android/firebase_messaging_cpp.aar"
59+
fi
5660
done
5761
cd "${origpath}"

0 commit comments

Comments
 (0)