Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI: remove .framework archives from releases #3582

Merged
merged 1 commit into from
Feb 7, 2024

Conversation

NachoSoto
Copy link
Contributor

@NachoSoto NachoSoto commented Jan 17, 2024

Carthage cannot build fat .frameworks on Apple Silicon:

require XCFrameworks when building on an Apple Silicon Mac

It fails with this error:

RevenueCat have the same architectures (arm64) and can't be in the same fat output file

To address that, we're planning to remove these on the next major release.
Instead, .xcframeworks, SPM, or CocoaPods should be used.

@NachoSoto NachoSoto added do not merge breaking Changes that are breaking labels Jan 17, 2024
@NachoSoto NachoSoto requested a review from a team January 17, 2024 20:37
@NachoSoto NachoSoto force-pushed the remove-carthage-archive branch 2 times, most recently from 5e76a83 to 7b213ea Compare January 17, 2024 21:21
@NachoSoto NachoSoto force-pushed the nacho/testing-circleci-m1 branch 2 times, most recently from f4a690a to 6dbe2bc Compare January 17, 2024 23:41
Base automatically changed from nacho/testing-circleci-m1 to main January 18, 2024 00:19
@NachoSoto NachoSoto force-pushed the remove-carthage-archive branch 2 times, most recently from 8300ac1 to 56198c2 Compare January 18, 2024 00:20
@@ -86,7 +86,7 @@ platform :ios do
repo_name: repo_name,
github_api_token: ENV["GITHUB_TOKEN"],
changelog_latest_path: changelog_latest_path,
upload_assets: ['RevenueCat.framework.zip', 'RevenueCat.xcframework.zip']
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure if a lot of people are using those but just in case we should mention this in the changelog once we release this

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yup:

remove .framework archives from releases

Also this is breaking so it'll be highlighted as a breaking change.

@NachoSoto NachoSoto force-pushed the remove-carthage-archive branch 2 times, most recently from d8a5fdf to 0177c1c Compare February 1, 2024 20:11
@NachoSoto NachoSoto mentioned this pull request Feb 7, 2024
@MarkVillacampa MarkVillacampa changed the base branch from main to 5.0-dev February 7, 2024 17:57
@NachoSoto
Copy link
Contributor Author

@MarkVillacampa feel free to merge this too

@MarkVillacampa MarkVillacampa enabled auto-merge (squash) February 7, 2024 18:51
@MarkVillacampa MarkVillacampa merged commit 6bd1169 into 5.0-dev Feb 7, 2024
24 checks passed
@MarkVillacampa MarkVillacampa deleted the remove-carthage-archive branch February 7, 2024 19:01
NachoSoto added a commit that referenced this pull request Feb 7, 2024
NachoSoto added a commit that referenced this pull request Feb 9, 2024
[`Carthage` cannot build fat `.framework`s on Apple
Silicon](https://github.com/Carthage/Carthage?tab=readme-ov-file#migrating-a-project-from-framework-bundles-to-xcframeworks):
> require `XCFrameworks` when building on an Apple Silicon Mac

It fails with this error:
> RevenueCat have the same architectures (arm64) and can't be in the
same fat output file

To address that, we're planning to remove these on the next major
release.
Instead, `.xcframework`s, SPM, or CocoaPods should be used.
NachoSoto added a commit that referenced this pull request Feb 9, 2024
NachoSoto added a commit that referenced this pull request Feb 12, 2024
[`Carthage` cannot build fat `.framework`s on Apple
Silicon](https://github.com/Carthage/Carthage?tab=readme-ov-file#migrating-a-project-from-framework-bundles-to-xcframeworks):
> require `XCFrameworks` when building on an Apple Silicon Mac

It fails with this error:
> RevenueCat have the same architectures (arm64) and can't be in the
same fat output file

To address that, we're planning to remove these on the next major
release.
Instead, `.xcframework`s, SPM, or CocoaPods should be used.
NachoSoto added a commit that referenced this pull request Feb 12, 2024
NachoSoto added a commit that referenced this pull request Feb 12, 2024
[`Carthage` cannot build fat `.framework`s on Apple
Silicon](https://github.com/Carthage/Carthage?tab=readme-ov-file#migrating-a-project-from-framework-bundles-to-xcframeworks):
> require `XCFrameworks` when building on an Apple Silicon Mac

It fails with this error:
> RevenueCat have the same architectures (arm64) and can't be in the
same fat output file

To address that, we're planning to remove these on the next major
release.
Instead, `.xcframework`s, SPM, or CocoaPods should be used.
NachoSoto added a commit that referenced this pull request Feb 12, 2024
NachoSoto added a commit that referenced this pull request Feb 12, 2024
NachoSoto added a commit that referenced this pull request Feb 14, 2024
[`Carthage` cannot build fat `.framework`s on Apple
Silicon](https://github.com/Carthage/Carthage?tab=readme-ov-file#migrating-a-project-from-framework-bundles-to-xcframeworks):
> require `XCFrameworks` when building on an Apple Silicon Mac

It fails with this error:
> RevenueCat have the same architectures (arm64) and can't be in the
same fat output file

To address that, we're planning to remove these on the next major
release.
Instead, `.xcframework`s, SPM, or CocoaPods should be used.
NachoSoto added a commit that referenced this pull request Feb 14, 2024
MarkVillacampa pushed a commit that referenced this pull request Feb 21, 2024
[`Carthage` cannot build fat `.framework`s on Apple
Silicon](https://github.com/Carthage/Carthage?tab=readme-ov-file#migrating-a-project-from-framework-bundles-to-xcframeworks):
> require `XCFrameworks` when building on an Apple Silicon Mac

It fails with this error:
> RevenueCat have the same architectures (arm64) and can't be in the
same fat output file

To address that, we're planning to remove these on the next major
release.
Instead, `.xcframework`s, SPM, or CocoaPods should be used.
MarkVillacampa pushed a commit that referenced this pull request Feb 21, 2024
MarkVillacampa pushed a commit that referenced this pull request Feb 21, 2024
joshdholtz pushed a commit that referenced this pull request Feb 23, 2024
[`Carthage` cannot build fat `.framework`s on Apple
Silicon](https://github.com/Carthage/Carthage?tab=readme-ov-file#migrating-a-project-from-framework-bundles-to-xcframeworks):
> require `XCFrameworks` when building on an Apple Silicon Mac

It fails with this error:
> RevenueCat have the same architectures (arm64) and can't be in the
same fat output file

To address that, we're planning to remove these on the next major
release.
Instead, `.xcframework`s, SPM, or CocoaPods should be used.
joshdholtz pushed a commit that referenced this pull request Feb 23, 2024
MarkVillacampa pushed a commit that referenced this pull request Mar 6, 2024
[`Carthage` cannot build fat `.framework`s on Apple
Silicon](https://github.com/Carthage/Carthage?tab=readme-ov-file#migrating-a-project-from-framework-bundles-to-xcframeworks):
> require `XCFrameworks` when building on an Apple Silicon Mac

It fails with this error:
> RevenueCat have the same architectures (arm64) and can't be in the
same fat output file

To address that, we're planning to remove these on the next major
release.
Instead, `.xcframework`s, SPM, or CocoaPods should be used.
MarkVillacampa pushed a commit that referenced this pull request Mar 6, 2024
MarkVillacampa pushed a commit that referenced this pull request Mar 6, 2024
[`Carthage` cannot build fat `.framework`s on Apple
Silicon](https://github.com/Carthage/Carthage?tab=readme-ov-file#migrating-a-project-from-framework-bundles-to-xcframeworks):
> require `XCFrameworks` when building on an Apple Silicon Mac

It fails with this error:
> RevenueCat have the same architectures (arm64) and can't be in the
same fat output file

To address that, we're planning to remove these on the next major
release.
Instead, `.xcframework`s, SPM, or CocoaPods should be used.
MarkVillacampa pushed a commit that referenced this pull request Mar 6, 2024
MarkVillacampa pushed a commit that referenced this pull request Apr 9, 2024
[`Carthage` cannot build fat `.framework`s on Apple
Silicon](https://github.com/Carthage/Carthage?tab=readme-ov-file#migrating-a-project-from-framework-bundles-to-xcframeworks):
> require `XCFrameworks` when building on an Apple Silicon Mac

It fails with this error:
> RevenueCat have the same architectures (arm64) and can't be in the
same fat output file

To address that, we're planning to remove these on the next major
release.
Instead, `.xcframework`s, SPM, or CocoaPods should be used.
MarkVillacampa pushed a commit that referenced this pull request Apr 9, 2024
fire-at-will pushed a commit that referenced this pull request Apr 15, 2024
[`Carthage` cannot build fat `.framework`s on Apple
Silicon](https://github.com/Carthage/Carthage?tab=readme-ov-file#migrating-a-project-from-framework-bundles-to-xcframeworks):
> require `XCFrameworks` when building on an Apple Silicon Mac

It fails with this error:
> RevenueCat have the same architectures (arm64) and can't be in the
same fat output file

To address that, we're planning to remove these on the next major
release.
Instead, `.xcframework`s, SPM, or CocoaPods should be used.
fire-at-will pushed a commit that referenced this pull request Apr 15, 2024
fire-at-will pushed a commit that referenced this pull request May 13, 2024
[`Carthage` cannot build fat `.framework`s on Apple
Silicon](https://github.com/Carthage/Carthage?tab=readme-ov-file#migrating-a-project-from-framework-bundles-to-xcframeworks):
> require `XCFrameworks` when building on an Apple Silicon Mac

It fails with this error:
> RevenueCat have the same architectures (arm64) and can't be in the
same fat output file

To address that, we're planning to remove these on the next major
release.
Instead, `.xcframework`s, SPM, or CocoaPods should be used.
fire-at-will pushed a commit that referenced this pull request May 13, 2024
fire-at-will pushed a commit that referenced this pull request Jun 24, 2024
fire-at-will pushed a commit that referenced this pull request Jun 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking Changes that are breaking
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants