search

risechain / pevm

Blazingly fast Parallel EVM
MIT License
232 stars 47 forks source link

feat: downgrade deps to make it compatible to reth 1.1.0 #403

Closed kien-rise closed 4 weeks ago

kien-rise commented 4 weeks ago

We need these changes to make sure there is only one version of alloy-eip7702 in the reth integration. Otherwise, there will be compile errors.

- op-alloy-consensus = { version = "0.5.0", optional = true }
- op-alloy-network = { version = "0.5.0", optional = true }
- op-alloy-rpc-types = { version = "0.5.0", optional = true }
+ op-alloy-consensus = { version = "0.4.0", optional = true }
+ op-alloy-network = { version = "0.4.0", optional = true }
+ op-alloy-rpc-types = { version = "0.4.0", optional = true }

This change is to update revm to the reth branch:

- revm = { git = "https://github.com/risechain/revm", rev = "bb350a971e5f39a8f822ace081584a3a45fd7c6a", features = [
+ revm = { git = "https://github.com/risechain/revm", rev = "fa685fbacc56718bdcc8aa11f851e5070c2d63b3", features = [

The other changes are just to make the code compiles.

Yes, the target branch of this PR is reth-integration-2 because those changes are not needed on main.

PS: Actually, if we want, we can keep op-alloy-* changes on main branch and revm change on this branch (i.e. split this PR into two PRs). We can do that to keep this branch minimal.

hai-rise commented 4 weeks ago

PS: Actually, if we want, we can keep op-alloy-* changes on main branch and revm change on this branch (i.e. split this PR into two PRs). We can do that to keep this branch minimal.

Nah, that's fine. Our end goal is to remove revm completely (#382) but keep Alloy (up-to-date) for primitive types, so this PR is in the right direction already. Once the old reth-integration is not needed anymore I'll squash this branch and force-push to it and keep this -2 as the new backup.