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

Fixing Carthage examples embed settings #827

Merged
merged 2 commits into from
May 24, 2021

Conversation

julepka
Copy link
Contributor

@julepka julepka commented May 24, 2021

  • Changes Do not embed to Embed and sign option for Carthage iOS for Swift and Objective-C
  • Update Carthage iOS Swift example show a Hello label when app is running
  • Update Carthage iOS Obj-C example to contain app icon to make it easier to test in TestFlight

Checklist

  • Change is covered by automated tests
  • The coding guidelines are followed
  • Public API has proper documentation
  • Example projects and code samples are up-to-date (in case of API changes)
  • Changelog is updated (in case of notable or breaking changes) - No new features added

@julepka julepka requested review from ilammy and vixentael May 24, 2021 13:43
Copy link
Collaborator

@ilammy ilammy left a comment

Choose a reason for hiding this comment

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

It's weird that the frameworks were not embedded before...

We have way too many examples to test them all manually.

@julepka julepka merged commit b8e024f into cossacklabs:master May 24, 2021
@julepka julepka added M-Carthage Package manager: Carthage, Objective-C and Swift, iOS and macOS O-iOS 📱 Operating system: iOS labels May 24, 2021
@julepka
Copy link
Contributor Author

julepka commented May 24, 2021

It's weird that the frameworks were not embedded before...

Previously there were a workaround for Carthage to copy frameworks. Like, you do not embed but it copies with a script or something like that... And with XCF support they've removed the workaround but I forgot to embed the framework 😓

@vixentael
Copy link
Contributor

We have way too many examples to test them all manually.

Truly so, the problem arise with testing examples on a real device / via TestFlight. We might want to add some extra step on archiving / uploading to TF, but we will still have a step to manually install on a real device.

julepka added a commit to julepka/themis that referenced this pull request May 26, 2021
julepka added a commit to julepka/themis that referenced this pull request May 26, 2021
julepka added a commit to julepka/themis that referenced this pull request May 26, 2021
vixentael pushed a commit that referenced this pull request May 26, 2021
* Cherry-pick: Updated Carthage examples to use XCF (#823)

* Fixing Carthage examples embed settings (#827)

* fix changelog conflict
vixentael pushed a commit that referenced this pull request May 26, 2021
* Updated podspec to support OpenSSL-XCF

* versions update

* update changelog

* Cherry-pick: Updated Carthage examples to use XCF (#823)

* Fixing Carthage examples embed settings (#827)

* Revert "Fixing Carthage examples embed settings (#827)"

This reverts commit d9403c5.

* Revert "Cherry-pick: Updated Carthage examples to use XCF (#823)"

This reverts commit 6c14677.

* update changelog

* added deprecation notice to podspec and changelog

* update changelog
julepka added a commit that referenced this pull request May 27, 2021
* Carthage XCF support and OpenSSL 1.1.1k (#817)
* Update Carthage examples to unbreak tests (#829)
* Cherry-pick: Updated Carthage examples to use XCF (#823)
* Fixing Carthage examples embed settings (#827)
* fix changelog conflict
* Themis CocoaPods now support CLOpenSSL-XCF (#828)
* Updated podspec to support OpenSSL-XCF
* versions update
* update changelog
* Cherry-pick: Updated Carthage examples to use XCF (#823)
* Fixing Carthage examples embed settings (#827)
* Revert "Fixing Carthage examples embed settings (#827)"
This reverts commit d9403c5.
* Revert "Cherry-pick: Updated Carthage examples to use XCF (#823)"
This reverts commit 6c14677.
* update changelog
* added deprecation notice to podspec and changelog
* update changelog
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
M-Carthage Package manager: Carthage, Objective-C and Swift, iOS and macOS O-iOS 📱 Operating system: iOS
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants