GHA: add Android builds for firebase-cpp-sdk #21
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
The C++ Interop efforts in Swift currently have some limitations. In particular, it cannot support trivial types with non-trivial destructors. As a workaround, provide a copy constructor which can be used by the Swift interop while using the regular semantics for all other cases. A second issue arises in the handling of futures. Unfortunately, it is not currently possible to pass an indirect block parameter which prevents the construction of a callback. Workaround this by providing an inline shim to use a direct parameter (i.e. indirect value through a pointer) which then allows a callback to be formed. Both of these items are being tracked upstream but seem to be potentially sufficient to enable the use of Swift for using the C++ SDK for desktop scenarios.
This adds a minimal build of firebase statically for use with Swift on Windows to provide the required base libraries. We build a release only build intended for re-distribution as a basis for building a library to access the Firebase APIs from Swift.
Fix repository URL
Correct enclosing namespace for `Firestore`.
Distribute firestore dependency headers
Enable support for ARM64. Add a matrix to allow building the x64 and ARM64 slices of the SDK.
Indicate that the build is performed in CI, explicitly disable the static MSVC library linkage.
Use explicit path to python interpreter.
Ensure that we use the same python version across the sub-builds as `Python3_EXECUTABLE` can be explicitly passed to control the version of python in use.
We would like to cross-compile to ARM64, so we need to ensure that we disable the flatc build.
Prebuild flatc manually and use that to build the flatbuffer code. This should help us with the cross-compilation for ARM64.
This pins the flatc build to the same revision that will be used in the build of firebase.
Make some pdbs
Enable CMP0141 to allow us to embed debug information for Firebase.
Reference some types so that they are included in the CXX module.
This introduces an initial nuget packaging for simpler distribution of the artifacts.
This will publish the package to ease the consumption of it in the client.
Remove an extraneous `include` in the path.
Add missing `firebase_util.lib` to the packaging. Thanks to @jeffdav for identifying the missing library.
Correct the library name to `firestore_util.lib`.
…Bs in the BCNY windows build (#20) * [windows][build] add a helper script to adjust MSVC debug flags to be Z7 instead of Zi * [bcny][gh workflow] build firebase with /Z7 for embedded debug information on windows * roll back to the mainline branch
compnerd
force-pushed
the
compnerd/android
branch
7 times, most recently
from
8a3464c
to
2449ca9
Compare
Introduce the Android builds so that we could start exploring the use of firebase on Android native code.
compnerd
force-pushed
the
compnerd/android
branch
from
2449ca9
to
9e15887
Compare
kendalharland
force-pushed
the
compnerd/swift
branch
from
46990b8
to
08b0c11
Compare
github-actions
bot
force-pushed
the
compnerd/swift
branch
from
1917cf3
to
705bb91
Compare
bcny-fork-syncer
bot
force-pushed
the
compnerd/swift
branch
2 times, most recently
from
e446911
to
13bdcdb
Compare
bcny-fork-syncer
bot
force-pushed
the
compnerd/swift
branch
2 times, most recently
from
573b8be
to
23e1651
Compare
bcny-fork-syncer
bot
force-pushed
the
compnerd/swift
branch
2 times, most recently
from
bdb199c
to
6171d35
Compare
bcny-fork-syncer
bot
force-pushed
the
compnerd/swift
branch
2 times, most recently
from
1b37356
to
43e9ddc
Compare
bcny-fork-syncer
bot
force-pushed
the
compnerd/swift
branch
from
c38940e
to
9f20338
Compare
bcny-fork-syncer
bot
force-pushed
the
compnerd/swift
branch
3 times, most recently
from
847a826
to
8d73af1
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Introduce the Android builds so that we could start exploring the use of firebase on Android native code.