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

fix cocoapods and carthage release scripts #324

Merged
merged 3 commits into from
Sep 1, 2020

Conversation

aboedo
Copy link
Member

@aboedo aboedo commented Aug 28, 2020

Cocoapods

pod trunk push failed because for pre-release versions, you need to specify the exact version number, and we had 3.6.0-beta-2.
I finished the process manually, but this gets it fixed for next time.

Carthage

Also fixes carthage.
carthage build --archive would fail, because it was trying to create two archives (one for Purchases and one for PurchasesCoreSwift.
Fixed by separating the shorthand carthage build --archive into

carthage build --no-skip-current
carthage archive Purchases

XCFrameworks

Turns out the xcframeworks fastlane plugin isn't compatible with static libraries, so the xcframeworks generation is broken.
image

We might want to roll it back to where it was before this commit:
3765e6f

And build from there, but that's for another day.

@aboedo aboedo requested a review from vegaro August 28, 2020 19:45
@aboedo aboedo self-assigned this Aug 28, 2020
@aboedo aboedo changed the title add explicit version to pre-release dependency fix cocoapods and carthage release scripts Aug 28, 2020
@@ -22,7 +22,7 @@ Pod::Spec.new do |s|
s.tvos.deployment_target = '9.0'

s.pod_target_xcconfig = { 'DEFINES_MODULE' => 'YES' }
s.dependency 'PurchasesCoreSwift'
s.dependency 'PurchasesCoreSwift', '3.6.0-beta-2'
Copy link
Member Author

Choose a reason for hiding this comment

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

I'll add this to the list of things for fastlane to auto-bump

@aboedo aboedo force-pushed the fix/add_explicit_version_to_prerelease_pod branch from b918c63 to 662d99d Compare August 31, 2020 20:24
@aboedo
Copy link
Member Author

aboedo commented Sep 1, 2020

@vegaro bump

@aboedo aboedo merged commit 993a981 into develop Sep 1, 2020
@aboedo aboedo deleted the fix/add_explicit_version_to_prerelease_pod branch September 1, 2020 22:04
@aboedo aboedo mentioned this pull request Sep 16, 2020
@aboedo aboedo mentioned this pull request Jul 13, 2021
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