-
Notifications
You must be signed in to change notification settings - Fork 225
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
wip: use libevm (params, core/vm) #1335
Draft
darioush
wants to merge
49
commits into
master
Choose a base branch
from
use-libevm-params
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Draft
Changes from all commits
Commits
Show all changes
49 commits
Select commit
Hold shift + click to select a range
6d60741
wip: use libevm (params)
darioush d999563
base change
darioush 1c7e25b
test unmarshal capability
darioush a119516
separate avalanche stuff to its own type
darioush 25bb446
separate RulesExtra
darioush c371a3a
separate calls to extra
darioush 9b27ebe
use darioush/access-list-for-hook
darioush 784e2e6
config_extra: move methods off Rules/ChainConfig receivers
darioush a8a5c80
un-embed RulesExtra
darioush d74cea6
params: use gethparams.Rules
darioush ed1165b
unused
darioush e1088b1
update chainConfig.Rules callsites to pass params.IsMergeTODO
darioush 846f52a
params/config: type alias ConfigCompatError
darioush 932967d
bring in gethparams.ChainConfig (wip)
darioush 053af95
params: add LondonBlock
darioush 44db6c7
params: add ShanghaiTime
darioush cd5adc9
params: add BerlinBlock
darioush 70659d3
params: unused code
darioush 5d5fb59
reduce diff
darioush 07203b0
update to darioush/json-root branch
darioush 99c1b70
params: add IsLondon checks to Cancun (similar to upstream)
darioush e8febb8
race free impl for WithExtra, still hack
darioush 04338c7
Merge branch 'master' of github.com:ava-labs/subnet-evm into use-libe…
darioush a5a43cc
use RulesExtra (instead of ptr)
darioush 4a41e81
bump libevm
darioush 5049989
bump libevm
darioush 2a46ead
chain hooks for create
darioush 672f54e
use geth forks in core/vm/evm.go
darioush 7b8ec21
core/types, core/state: import gethparams
darioush f30f450
core/vm: use gethparams and hook in precompiles
darioush dc4b361
core/types: use upstream Log type
darioush 5038b15
core/types: use upstream for AccessList
darioush dad843a
core/vm: use libevm
darioush 0fe10ff
linting
darioush 5b073ad
fix mockgen
darioush 34a17a6
adjustment for libevm chainconfig changes
darioush 73b5559
populate vm.BlockContext.Random as Difficulty starting at Durango (to…
darioush 61b1694
nit
darioush 8d2493f
Merge branch 'master' of github.com:ava-labs/subnet-evm into use-libe…
darioush 12cd83e
update libevm
darioush 20854a1
update libevm
darioush e9ee414
rebase to latest libevm (+ fix commit on darioush/json-root)
darioush 61eb933
add support for custom upstream tag
darioush 28caecc
remove eth/tracers subdirs
darioush 86cdfbe
update geth-allowed-packages.txt
darioush f1d77d2
Merge branch 'master' of github.com:ava-labs/subnet-evm into use-libe…
darioush 7bfbe08
lint
darioush 27a8bfb
more lint
darioush d72ff25
more lint
darioush File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -2179,6 +2179,11 @@ func golangBindings(t *testing.T, overload bool) { | |
if out, err := replacer.CombinedOutput(); err != nil { | ||
t.Fatalf("failed to replace binding test dependency to current source tree: %v\n%s", err, out) | ||
} | ||
replacer = exec.Command(gocmd, "mod", "edit", "-x", "-require", "github.com/ethereum/[email protected]", "-replace", "github.com/ethereum/go-ethereum=github.com/ava-labs/[email protected]") | ||
replacer.Dir = pkg | ||
if out, err := replacer.CombinedOutput(); err != nil { | ||
t.Fatalf("failed to replace binding test dependency to current source tree: %v\n%s", err, out) | ||
} | ||
tidier := exec.Command(gocmd, "mod", "tidy", "-compat=1.21") | ||
tidier.Dir = pkg | ||
if out, err := tidier.CombinedOutput(); err != nil { | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -695,6 +695,11 @@ func TestPrecompileBind(t *testing.T) { | |
if out, err := replacer.CombinedOutput(); err != nil { | ||
t.Fatalf("failed to replace binding test dependency to current source tree: %v\n%s", err, out) | ||
} | ||
replacer = exec.Command(gocmd, "mod", "edit", "-x", "-require", "github.com/ethereum/[email protected]", "-replace", "github.com/ethereum/go-ethereum=github.com/ava-labs/[email protected]") | ||
replacer.Dir = pkg | ||
if out, err := replacer.CombinedOutput(); err != nil { | ||
t.Fatalf("failed to replace binding test dependency to current source tree: %v\n%s", err, out) | ||
} | ||
tidier := exec.Command(gocmd, "mod", "tidy", "-compat=1.21") | ||
tidier.Dir = pkg | ||
if out, err := tidier.CombinedOutput(); err != nil { | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -37,13 +37,13 @@ import ( | |
"github.com/ava-labs/subnet-evm/consensus/dummy" | ||
"github.com/ava-labs/subnet-evm/core/state" | ||
"github.com/ava-labs/subnet-evm/core/types" | ||
"github.com/ava-labs/subnet-evm/core/vm" | ||
"github.com/ava-labs/subnet-evm/eth/tracers" | ||
"github.com/ava-labs/subnet-evm/eth/tracers/logger" | ||
"github.com/ava-labs/subnet-evm/params" | ||
"github.com/ava-labs/subnet-evm/tests" | ||
"github.com/ethereum/go-ethereum/common" | ||
"github.com/ethereum/go-ethereum/common/hexutil" | ||
"github.com/ethereum/go-ethereum/core/vm" | ||
"github.com/ethereum/go-ethereum/eth/tracers/logger" | ||
"github.com/ethereum/go-ethereum/log" | ||
"github.com/urfave/cli/v2" | ||
) | ||
|
@@ -205,7 +205,7 @@ func Transition(ctx *cli.Context) error { | |
|
||
func applyLondonChecks(env *stEnv, chainConfig *params.ChainConfig) error { | ||
// NOTE: IsLondon replaced with IsSubnetEVM here | ||
if !chainConfig.IsSubnetEVM(env.Timestamp) { | ||
if !params.GetExtra(chainConfig).IsSubnetEVM(env.Timestamp) { | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I wonder if we should just use London here |
||
return nil | ||
} | ||
// Sanity check, to not `panic` in state_transition | ||
|
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
any reason why we not directly use ava-labs/go-ethereum as directy dependency?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The module renaming is not ready yet, we will use it directly once it is ready.