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

Restore the getrandom feature in rust-runtime.a static lib #1563

Closed
wants to merge 3 commits into from

Conversation

nategraf
Copy link
Contributor

Restore the getrandom feature in rust-runtime.a static lib. This static library build primarily supports the cargo risczero test tool, which we use in our workflows to test patched cryptography crates and potentially other crates developed for the zkVM. getrandom is intended to be supported, but was removed with the change of default behavior in #1162.

Copy link

vercel bot commented Mar 15, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
website ✅ Ready (Inspect) Visit Preview Mar 15, 2024 11:27pm

Base automatically changed from victor/fix-execute-in-dev-mode to main March 18, 2024 18:04
@SchmErik
Copy link
Contributor

@nategraf it looks like you have some other changes in addition to the getrandom feature

@nategraf
Copy link
Contributor Author

Sorry, yes. I was working on getting cargo risczero test to work, which is where the null segment ref changes come in. I'll refactor/clean this up tomorrow. I do want to land all the included changes, but the description needs to reflect the contents, and possibly two PRs are in order.

@flaub
Copy link
Member

flaub commented Apr 24, 2024

I think we need to get some CI around this staticlib. Ideally we'd run cargo risczero test somewhere in CI

@SchmErik SchmErik self-assigned this Apr 29, 2024
@SchmErik
Copy link
Contributor

superseded by #1750. closing

@SchmErik SchmErik closed this Apr 30, 2024
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

3 participants