-
Notifications
You must be signed in to change notification settings - Fork 212
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
Integrate MatrixSDKCrypto #1501
Conversation
@@ -22,12 +22,12 @@ Pod::Spec.new do |s| | |||
s.requires_arc = true | |||
s.swift_versions = ['5.1', '5.2'] | |||
|
|||
s.ios.deployment_target = "10.0" | |||
s.ios.deployment_target = "11.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Need to increase deployment target everywhere to 11, because 10 still supports 32bit architectures, whereas Rust has already dropped support for these architectures so MatrixCryptoSDK would be incompatible.
private let machine: OlmMachine | ||
public override init() { | ||
do { | ||
machine = try OlmMachine(userId: "", deviceId: "", path: "", passphrase: nil) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
OlmMachine
is a symbol defined in MatrixSDKCrypto
pod
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We need an .api
changelog file to communicate on the iOS 10 drop. Then, it will be all good for me.
Exciting time!
Is this in addition or instead of the existing |
In addition, you can have several changelog files in a PR. |
Builds on top of experimental
MXCryptoV2
and integrates independently publishedMatrixSDKCrypto
CocoaPod, published as offical CocoaPodTo avoid bloating up the size of production builds as well as making them safer, the new pod will only be loaded in DEBUG builds and is currently compiled only for iOS, as some of the internal libraries in Rust Crypto cannot be compiled for macos at the moment. This will be resolved in the future but does not have to be a blocker at this point.
Finally note that the instantiation of
OlmMachine
in this PR means the integration of the pod works as expected.