Skip to content

Conversation

@Loupehope
Copy link
Contributor

This pull-request adds static framework generation on CI.

Fixes: #2485 and #2486

Additionally later, can be added to https://github.com/airbnb/lottie-spm

I would be grateful if you could double-check the code, I don't work with ruby that often.

Example output of a type of framework for dynamic linking:
Screenshot 2025-05-09 at 18 51 49

Example of framework type output for static linking:
Screenshot 2025-05-09 at 18 52 17

@calda
Copy link
Member

calda commented May 9, 2025

What use case do you have in mind for a static XCFramework?

@Loupehope
Copy link
Contributor Author

Loupehope commented May 9, 2025

We use Tuist for development, which suggests using a static framework for SPM dependencies by default - https://docs.tuist.dev/en/guides/develop/projects/dependencies#tuists-xcodeprojbased-integration.

In addition, the release application is built in static to avoid wasting time loading dynamic libraries on startup.

It seemed to me that it would be convenient for such a set-up to have a static framework in addition to the pre-built dynamic framework.

Artifact download URL: https://github.com/airbnb/lottie-ios/actions/runs/14942555625/artifacts/3098429181

@calda
Copy link
Member

calda commented May 10, 2025

Sounds good, thanks for the PR

@calda calda enabled auto-merge (squash) May 10, 2025 16:57
@calda calda disabled auto-merge May 10, 2025 16:57
@calda calda merged commit d6ddd82 into airbnb:master May 10, 2025
11 of 12 checks passed
@Loupehope Loupehope deleted the feature/github-actions-static-framework branch May 25, 2025 12:52
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.

[FR] Add a static xcframework to tags.

2 participants