-
Notifications
You must be signed in to change notification settings - Fork 84
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
Support native touchid devices #207
Comments
@micolous It sounds like @ericmarkmartin is keen to have a look at this, and it seems to overlap/be similar to your work with the windows apis. |
Cool! I've not looked into it a huge amount, so I'll hold off for now. Here is a good starting point:
I think what'd need to happen is pretty similar to what I did with the Windows stuff -- map all the |
Do we have strong feelings about using an Objc FFI over Swift bindings? Afaik, the rust-objc and swift-bridge crates are the standard for objc and swift bindings in rust respectively. |
@ericmarkmartin given the way apple is trending, swift might be the safer choice to use here, unless there is a compelling reason to go with obj-c? |
The only thing I can think of is that the rust objc binding foo might be more mature but I actually don't know enough to claim that for certain. |
I don't have strong feelings either way – I hadn't looked into it a whole lot yet. |
@ericmarkmartin Yeah, I think it'll be a case of investigating the two options and deciding :) |
I wouldn't use objC, it's the "legacy" option as far as Apple's concerned. |
I don't think we have much of an option. Support for interacting with swift from rust is quite limited. Afaik there's the swift-rs and swift-bridge. I think using the former would require us to write a lot of swift so we can expose the ASAuthorizationServices API with functions with I also don't think we're likely to run into an issue with Apple deprecating Objective C in some way. For newer APIs, all they're doing is writing the swift such that it's exposed to the objc runtime, and this is sufficiently low cost that I imagine they'll keep doing it. |
No description provided.
The text was updated successfully, but these errors were encountered: