Skip to content

Strip debug symbols from the Apple libraries#1443

Open
a-maurice wants to merge 4 commits intomainfrom
am-apple_stripping
Open

Strip debug symbols from the Apple libraries#1443
a-maurice wants to merge 4 commits intomainfrom
am-apple_stripping

Conversation

@a-maurice
Copy link
Copy Markdown
Collaborator

@a-maurice a-maurice commented Apr 27, 2026

Description

Provide details of the change, and generalize the change in the PR title above.

Reduce the size of the built Apple and Linux libraries by stripping unnecessary symbols from them.


Testing

Describe how you've tested these changes.

https://github.com/firebase/firebase-unity-sdk/actions/runs/25084056378


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.

Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request introduces symbol stripping for macOS builds in both build_shared.cmake and build_universal.cmake using linker options and post-build commands. Feedback suggests restricting this stripping to non-debug builds to preserve debuggability and using the ${CMAKE_STRIP} variable instead of a hardcoded command to ensure toolchain consistency.

Comment thread cmake/build_shared.cmake Outdated
Comment thread cmake/build_shared.cmake Outdated
Comment thread cmake/build_universal.cmake Outdated
Comment thread cmake/build_universal.cmake Outdated
@a-maurice a-maurice marked this pull request as ready for review April 30, 2026 00:07
@a-maurice a-maurice requested a review from AustinBenoit April 30, 2026 00:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants