Use llvm-objcopy to package Windows binaries by triplef · Pull Request #1427 · firebase/firebase-cpp-sdk
Description
Changes the packaging script to use LLVM binutils. This fixes object files corrupted by using objcopy by using llvm-objcopy instead. While the corrupted files were working fine when using the Visual Studio toolchain they were leading to errors when using the LLVM toolchain.
See #793 (comment) for details.
Testing
Needs to be run as part of the GitHub actions packaging. I will test builds once they are available.
Type of Change
Place an x the applicable box:
- Bug fix. Add the issue # below if applicable.
- New feature. A non-breaking change which adds functionality.
- Other, such as a build process or documentation change.
Notes
- Bug fixes and feature changes require an update to the
Release Notessection ofrelease_build_files/readme.md. - Read the contribution guidelines CONTRIBUTING.md.
- Changes to the public API require an internal API review. If you'd like to help us make Firebase APIs better, please propose your change in a feature request so that we can discuss it together.