-
Notifications
You must be signed in to change notification settings - Fork 150
zeroize: fixup rust 1.92 warnings #1270
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
Conversation
This changes the tests to highlight the new warnings
|
If possible, a zeroize_derive 1.5.1 release with that PR would be appreciated! |
|
@baloo hmm, 1.5.0 is still unreleased, and includes an MSRV bump We should probably get new releases of |
|
Ha, right! I didn't even check. Maybe I should backport this to 1.4 then. |
This is a backport of: - RustCrypto#1072 - RustCrypto#1270
|
I don't really have a way to open a PR for a backport because there is no branch to target to. I've made a commit:
If you could push a 1.4.3, that would be appreciated! I've checked that commit on rpgp (and 1.92), and it doesn't report any warnings anymore. |
|
@tarcieri do you plan on doing a |
|
Yeah, I'll take a look tomorrow |
|
a small release of zeroize itself (That bumps the dependency of zeroize_derive) would probably be nice to avoid issues with the minimal-versions checks as well. |
|
Well, zeroize 1.9 would imply an MSRV bump. Which isn’t something necessarily doable for someone using zeroize 1.8. this I why I was hinting at a zeroize 1.8.3 with a dependency on zeroize_derive 1.4.3. otherwise this forces consumers to have a direct dependency on zeroize_derive to force 1.4.3 (to fix the minimal-versions checks). This is at least the case in rpgp, but there might be other consumers. |
This fixes unhelpful warnings RustCrypto/utils#1270
This fixes unhelpful warnings RustCrypto/utils#1270
RustCrypto#1270 broke the tests for zeroize_derive, this fixes the expected output of the tests accordingly.
RustCrypto#1270 broke the tests for zeroize_derive, this fixes the expected output of the tests accordingly.
RustCrypto#1270 broke the tests for zeroize_derive, this fixes the expected output of the tests accordingly.
RustCrypto#1270 broke the tests for zeroize_derive, this fixes the expected output of the tests accordingly.
RustCrypto#1270 broke the tests for zeroize_derive, this fixes the expected output of the tests accordingly.
RustCrypto#1270 broke the tests for zeroize_derive, this fixes the expected output of the tests accordingly.
RustCrypto#1270 broke the tests for zeroize_derive, this fixes the expected output of the tests accordingly.
RustCrypto#1270 broke the tests for zeroize_derive, this fixes the expected output of the tests accordingly.
RustCrypto#1270 broke the tests for zeroize_derive, this fixes the expected output of the tests accordingly.
RustCrypto#1270 broke the tests for zeroize_derive, this fixes the expected output of the tests accordingly.
#[derive(ZeroizeOnDrop)]was throwing warnings like:This changes the tests to highlight the new warnings.