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

Release 1.4.3 #128

Merged
merged 6 commits into from
Nov 4, 2020
Merged

Release 1.4.3 #128

merged 6 commits into from
Nov 4, 2020

Conversation

aboedo
Copy link
Member

@aboedo aboedo commented Nov 4, 2020

1.4.3

  • Explicitly added Purchases framework to podspec's linker flags so it gets correctly linked in the plugin pod.
    Fixed mapping of xcconfig files in Runner example.
    Release 1.4.3 #128

@aboedo aboedo self-assigned this Nov 4, 2020
@aboedo aboedo marked this pull request as ready for review November 4, 2020 19:48
s.pod_target_xcconfig = {
'DEFINES_MODULE' => 'YES',
'EXCLUDED_ARCHS[sdk=iphonesimulator*]' => 'i386',
'OTHER_LDFLAGS' => '-framework Purchases'
Copy link
Member Author

Choose a reason for hiding this comment

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

this line is the actual fix. it adds a missing linker flag that links the Purchases framework.
I'm not sure why this flag isn't added automatically, I'm still figuring that out. But it did fix the linker issues on the test project I'm using

@aboedo aboedo requested a review from vegaro November 4, 2020 19:50
@@ -539,7 +539,7 @@
};
97C147071CF9000F007C117D /* Release */ = {
isa = XCBuildConfiguration;
baseConfigurationReference = 2FBF59F345E05927E16F325B /* Pods-Runner.release.xcconfig */;
baseConfigurationReference = 7AFA3C8E1D35360C0083082E /* Release.xcconfig */;
Copy link
Contributor

Choose a reason for hiding this comment

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

Any idea why this changed? It's just the example but just curious

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 should have added a comment, my bad.
This is a revert of a misguided update from the previous PR: https://github.com/RevenueCat/purchases-flutter/pull/127/files

Turns out that when you manually pod install, you get a warning from cocoapods, so I made the change on the previous pr to fix it...
But the warning was wrong.
Essentially the warning is cocoapods saying "you should be using the xcconfig from cocoapods", but the ones generated by flutter already include the ones in cocoapods.

@aboedo aboedo merged commit 1402657 into develop Nov 4, 2020
@aboedo aboedo deleted the release/1.4.3 branch November 4, 2020 20:21
Jethro87 pushed a commit to Jethro87/purchases-flutter that referenced this pull request Jan 4, 2025
* fixed xcconfig files mapping

* explicitly added s.framework = 'Purchases' so cocoapods links the Purchases target correctly in flutter projects.

* bumped version number to 1.4.3

* moved from framework to ldflag

* updated changelog

* clean up pod_target_xcconfig
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