-
Notifications
You must be signed in to change notification settings - Fork 61
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
Add capability to load symbols from current executable #230
Conversation
Signed-off-by: Elise Chouleur <[email protected]>
5d82618
to
9b35715
Compare
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.
Thanks for the PR!
I am curious, in which context would the library be contained in the executable? In case it has been statically linked 🤔?
Also wondering if instead of changing the interface and introducing the new LibLoadingType
we could create a new constructor, like new_from_self
which would do the same :) What do you think?
I agree with @hug-dev that this can be simplified. Btw, I'm on mobile, isn't the libloading dep missing from cargo.toml? The idea behind the change is good though! |
Signed-off-by: Elise Chouleur <[email protected]>
Yes when it's statically linked, which is mandatory on iOS when following the AppStore rules :) |
We were already using this on iOS and Android with the old rust-pkcs11 crate (but never merged on upstream), so it's working well. And I've tested it on a real device ;) |
Signed-off-by: Elise Chouleur <[email protected]>
249e170
to
1e65dd3
Compare
This is shaping really nicely 👌 I won't be able to look at it deeply before Monday, though :( |
Signed-off-by: Elise Chouleur <[email protected]>
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.
LGTM 👍 thanks!
Could you tell me what's happening next ? Or more precisely, what could be the ETA of an official release with this version ? :) |
We have no schedule set! It's all based on demand. Since the last release was in June and we got quite a lot of contributions recently, I guess we could make a new version if you wanted! |
I think it's a good idea. As you've said we have quite a lot of good contributions piled-up in |
We can maybe announce in the community channel that a new release will be done in like two weeks, so that people have time to push in more changes before the release? |
Yes, excellent idea! :) |
If PKCS#11 symbols are available inside the current executable as it can be necessary for iOS development.