a16z / jolt

The simplest and most extensible zkVM. Fast and fully open source from a16z crypto and friends. ⚡
https://jolt.a16zcrypto.com
MIT License
580 stars 105 forks source link

Fix tokio — this fixes latest compile errors in CI #396

Closed imikushin closed 2 weeks ago

imikushin commented 2 weeks ago

This fixes errors that have started popping up in clippy and test CI runs today:

    Checking jolt-core v0.1.0 (/home/runner/work/jolt/jolt/jolt-core)
error[E0599]: no function or associated item named `new` found for struct `tokio::runtime::Runtime` in the current scope
Error:    --> jolt-core/src/host/toolchain.rs:24:27
    |
24  |         let rt = Runtime::new().unwrap();
    |                           ^^^ function or associated item not found in `Runtime`
    |
note: if you're trying to build a new `tokio::runtime::Runtime`, consider using `tokio::runtime::Runtime::from_parts` which returns `tokio::runtime::Runtime`
   --> /home/runner/.cargo/registry/src/index.crates.io-6f17d22bba15001f/tokio-1.38.0/src/runtime/runtime.rs:138:5
    |
138 | /     pub(super) fn from_parts(
139 | |         scheduler: Scheduler,
140 | |         handle: Handle,
141 | |         blocking_pool: BlockingPool,
142 | |     ) -> Runtime {
    | |________________^
    = help: items from traits can only be used if the trait is implemented and in scope
note: `field::binius::BiniusConstructable` defines an item `new`, perhaps you need to implement it
   --> jolt-core/src/field/binius.rs:25:1
    |
25  | pub trait BiniusConstructable {
    | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

For more information about this error, try `rustc --explain E0599`.
error: could not compile `jolt-core` (lib) due to 1 previous error
Error: The process '/home/runner/.cargo/bin/cargo' failed with exit code 101
imikushin commented 2 weeks ago

@moodlezoup Noticed the above nastiness showing up in CI.

mw2000 commented 2 weeks ago

@moodlezoup I'm facing the same in CI for #385