-
Notifications
You must be signed in to change notification settings - Fork 54
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
Updated the tpm2-tss version for bindings. #505
Conversation
I'm in favor of this PR but I've noticed that the bindgen versions need to somehow align with parsec: parallaxsecond/rust-cryptoki#199 (comment) (I'm not saying there's a conflict, just noticed a similar PR in cryptoki). |
f89104e
to
1451c51
Compare
I have removed that for now. It can be changed at a later stage if necessary. |
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.
Well... it's bunch of generated code so there's not much to review here anyway :)
8e4c9f5
to
8ab0de2
Compare
- Generate the bindings from version 3.2.2 of the tpm2-tss library. - Add comment in the generated bindings from what version it was generated. - Starts using an explicit 4.0.1 version in tests. Signed-off-by: Jesper Brynolf <[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 👍
Fixes #496