-
Notifications
You must be signed in to change notification settings - Fork 58
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
Cargo.toml: tss-esapi bindings #502
Conversation
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 catching this; looks good to me (nit: a typo in commit message: "ammount" → "amount").
I will fix the commit message, but I will drop the changes in the lock file, I think that they should not be there |
b1b070d
to
df18e7f
Compare
Should me done, tested here: https://build.opensuse.org/package/show/home:aplanas:branches:security/rust-keylime and we have ppc and s390x binaries : ) |
cargo audit detects an issue that the unmaintained Maybe we need a bump of bindgen requirement in rust-tss-esapi to 0.60 at least? |
The optional bindgen dependency was set to a version that depends on an old version of clap, and also depends on the ansi_term crate, now unmaintained and causing audit complains. See [1] for an example. This patch upgrade to bindgen 0.63.0 and drop the dependency to the affected crate. [1] keylime/rust-keylime#502 Signed-off-by: Alberto Planas <[email protected]>
Done here: parallaxsecond/rust-tss-esapi#386 |
The optional bindgen dependency was set to a version that depends on an old version of clap, and also depends on the ansi_term crate, now unmaintained and causing audit complains. See [1] for an example. This patch upgrade to bindgen 0.60.0 and drop the dependency to the affected crate. In the future we should update 0.63, but in this case the minimal rustc version required will be 1.57 [1] keylime/rust-keylime#502 Signed-off-by: Alberto Planas <[email protected]>
The optional bindgen dependency was set to a version that depends on an old version of clap, and also depends on the ansi_term crate, now unmaintained and causing audit complains. See [1] for an example. This patch upgrade to bindgen 0.63.0 and drop the dependency to the affected crate. [1] keylime/rust-keylime#502 Signed-off-by: Alberto Planas <[email protected]>
Generate the tss-esapi bindings during compilation time. This will extend the amount of supported architectures. This is a regression done after the crate split, that was fixed before in keylime#459. Signed-off-by: Alberto Planas <[email protected]>
df18e7f
to
9b6a16a
Compare
/packit retest-failed |
The optional bindgen dependency was set to a version that depends on an old version of clap, and also depends on the ansi_term crate, now unmaintained and causing audit complains. See [1] for an example. This patch upgrade to bindgen 0.63.0 and drop the dependency to the affected crate. [1] keylime/rust-keylime#502 Signed-off-by: Alberto Planas <[email protected]>
Generate the tss-esapi bindings during compilation time. This will extend the ammount of supported architectures.
This is a regression done after the crate split, that was fixed before in #459.
Signed-off-by: Alberto Planas [email protected]