Skip to content

Implement EVM#214

Merged
dmitrylavrenov merged 44 commits intomasterfrom
pallet-evm
Jan 20, 2022
Merged

Implement EVM#214
dmitrylavrenov merged 44 commits intomasterfrom
pallet-evm

Conversation

@dmitrylavrenov
Copy link
Contributor

Closes #79

@dmitrylavrenov
Copy link
Contributor Author

@MOZGIII Some Ethereum primitives require the following function sp_core::to_substrate_wasm_fn_return_value;. But our current substrate fork doesn't contain it. For now it's a blocker to me.

We need to update our substrate fork.

@MOZGIII
Copy link
Contributor

MOZGIII commented Dec 3, 2021

Let's update it then, #205 is another reason to upgrade.

@MOZGIII
Copy link
Contributor

MOZGIII commented Dec 3, 2021

image

@dmitrylavrenov dmitrylavrenov marked this pull request as draft December 3, 2021 18:14
@MOZGIII MOZGIII changed the base branch from master to substrate-update December 14, 2021 13:15
@dmitrylavrenov
Copy link
Contributor Author

dmitrylavrenov commented Dec 15, 2021

Added evm configuration options as EthApi requires a parameter max_past_logs to be defined. @MOZGIII One question, should we provide a user an optional way to run with evm RPC api or without?

@MOZGIII
Copy link
Contributor

MOZGIII commented Dec 16, 2021

@MOZGIII One question, should we provide a user an optional way to run with evm RPC api or without?

Yes, this will be a required feature. We will also need a way to disable the bioauth API group.

Base automatically changed from substrate-update to master December 17, 2021 09:20
@MOZGIII
Copy link
Contributor

MOZGIII commented Jan 20, 2022

Regarding frontier upgrade - try updating ethereum crate (or wherever the access list type is imported from).

@dmitrylavrenov dmitrylavrenov marked this pull request as ready for review January 20, 2022 10:40
@dmitrylavrenov dmitrylavrenov merged commit 7d6e7e9 into master Jan 20, 2022
@dmitrylavrenov dmitrylavrenov deleted the pallet-evm branch January 20, 2022 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

Implement EVM

2 participants