Skip to content
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

zeroize v1.5.0 #713

Merged
merged 1 commit into from Jan 14, 2022
Merged

zeroize v1.5.0 #713

merged 1 commit into from Jan 14, 2022

Conversation

tarcieri
Copy link
Member

Added

  • Zeroize impls for PhantomData, PhantomPinned, and tuples with 0-10 elements (#660)
  • #[zeroize(bound = "T: MyTrait")] (#663)
  • ZeroizeOnDrop trait and custom derive (#699, #700, #703)

@tarcieri tarcieri merged commit 5092abf into master Jan 14, 2022
@tarcieri tarcieri deleted the zeroize/v1.5.0 branch January 14, 2022 19:52
@dovahcrow
Copy link

Spent half an hour debugging derive macro zeroize not found issue when compiling a Solana program which depends on tiny-bip which depends on zeroize then found out it is because I was caught in between the release process of zeroize. What a sad story for the deep dependencies of open-source projects.

@tarcieri
Copy link
Member Author

Please open an issue with a reproduction of your problem

@dovahcrow
Copy link

Please open an issue with a reproduction of your problem

It's fixed after running cargo update.

This crate is not working during the 30 mins period between the release of zeroize_derive 1.3.0 and zeroize 1.5.0. Unfortunately I did the cargo update right after zeroize_derive so everything breaks.

@tarcieri
Copy link
Member Author

The new release of zeroize_derive should still be compatible with older versions of zeroize. If not, it's a bug.

@raphaelrobert
Copy link

I ran into a similar problem. Right now it looks like older versions (<=1.4.3) of zeroize don't work with rustc 1.58, but I need to investigate more.

@tarcieri
Copy link
Member Author

If someone can post the exact error you're encountering that'd be very helpful

@daxpedda
Copy link
Contributor

@tarcieri feel free to CC me when we find out what the issue is.

@raphaelrobert
Copy link

@daxpedda see #714.

@dovahcrow
Copy link

@raphaelrobert @daxpedda @tarcieri added the repro to #714

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants