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

Bump ring version to 0.17 #5

Closed
DragonDev1906 opened this issue Nov 23, 2023 · 4 comments
Closed

Bump ring version to 0.17 #5

DragonDev1906 opened this issue Nov 23, 2023 · 4 comments

Comments

@DragonDev1906
Copy link
Contributor

I'd like to use ethereum_hashing (and more specifically tree_hash and ssz_types) for an application running on the x86_64-fortanix-unknown-sgx target. Unfortunately, the ring dependency version 0.16.x does not compile with this target, this was added last week (see the addition of the feature to make it work/compile, feature renaming and a comment mentioning this.

ring v0.16.20
└── ethereum_hashing v1.0.0-beta.2
    └── tree_hash v0.5.2
        └── ssz_types v0.5.4

As of now, ring does not have a release containing this commit, but this will likely be added in the next version (0.17.6). Can we bump the ring dependency to 0.17 or any 0.17.x? That would allow using the commit that contains this change and would allow using the newest ring version as soon as it is released (unless there also is a breaking change in ring).

@michaelsproul
Copy link
Member

Yeah this sounds good. I'm also inclined to say we should downgrade ethereum_hashing's version from a v1.0.0-beta, because this will be a breaking release. We already did this for the other crates that have non 1.0 dependencies, and I think it's the correct approach per Rust crate guidelines.

TL;DR: please PR a ring update and then I'll bump the version to something that isn't 1.0

@michaelsproul
Copy link
Member

New version 0.6.0 is out, thanks!

@michaelsproul
Copy link
Member

hey @DragonDev1906 I've also released tree_hash and ssz_types 0.6 now

@DragonDev1906
Copy link
Contributor Author

Awesome, thank you ❤️
Since ring has now released a version with the commit I've mentioned (17.6 and 17.7) I should be able to use these crates on x86_64-fortanix-unknown-sgx.

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

No branches or pull requests

2 participants