libevm renaming nits (#683)

Author: Darioush Jalali

.github/workflows/tests.yml

@@ -17,7 +17,7 @@ jobs:
     runs-on: ubuntu-20.04
     steps:
       - uses: actions/checkout@v4
-      - run: ./scripts/lint_allowed_geth_imports.sh
+      - run: ./scripts/lint_allowed_eth_imports.sh
         shell: bash
       - uses: actions/setup-go@v5
         with:

core/evm.go

@@ -37,7 +37,7 @@ import (
 	"github.com/ava-labs/coreth/params"
 	"github.com/ava-labs/coreth/predicate"
 	"github.com/ava-labs/libevm/common"
-	gethtypes "github.com/ava-labs/libevm/core/types"
+	ethtypes "github.com/ava-labs/libevm/core/types"
 	"github.com/ava-labs/libevm/core/vm"
 	"github.com/ava-labs/libevm/log"
 	"github.com/holiman/uint256"
@@ -152,7 +152,7 @@ func newEVMBlockContext(header *types.Header, chain ChainContext, author *common
 		BaseFee:     baseFee,
 		BlobBaseFee: blobBaseFee,
 		GasLimit:    header.GasLimit,
-		Header: &gethtypes.Header{
+		Header: &ethtypes.Header{
 			Number: new(big.Int).Set(header.Number),
 			Time:   header.Time,
 			Extra:  extra,

core/types/block.go

@@ -67,7 +67,7 @@ func (n *BlockNonce) UnmarshalText(input []byte) error {
 }
 
 //go:generate go run github.com/fjl/gencodec -type Header -field-override headerMarshaling -out gen_header_json.go
-//go:generate go run github.com/ethereum/go-ethereum/rlp/rlpgen -type Header -out gen_header_rlp.go
+//go:generate go run github.com/ava-labs/libevm/rlp/rlpgen -type Header -out gen_header_rlp.go
 
 // Header represents a block header in the Ethereum blockchain.
 type Header struct {

core/types/imports.go

@@ -4,74 +4,74 @@
 package types
 
 import (
-	gethtypes "github.com/ava-labs/libevm/core/types"
+	ethtypes "github.com/ava-labs/libevm/core/types"
 )
 
 // The following types are used directly as their upstream definitions.
 // So we list them all here to avoid having many individual files.
 type (
-	AccessList        = gethtypes.AccessList
-	AccessTuple       = gethtypes.AccessTuple
-	AccessListTx      = gethtypes.AccessListTx
-	Bloom             = gethtypes.Bloom
-	Receipt           = gethtypes.Receipt
-	Receipts          = gethtypes.Receipts
-	ReceiptForStorage = gethtypes.ReceiptForStorage
-	LegacyTx          = gethtypes.LegacyTx
-	DynamicFeeTx      = gethtypes.DynamicFeeTx
-	BlobTx            = gethtypes.BlobTx
-	BlobTxSidecar     = gethtypes.BlobTxSidecar
-	Signer            = gethtypes.Signer
-	HomesteadSigner   = gethtypes.HomesteadSigner
-	FrontierSigner    = gethtypes.FrontierSigner
+	AccessList        = ethtypes.AccessList
+	AccessTuple       = ethtypes.AccessTuple
+	AccessListTx      = ethtypes.AccessListTx
+	Bloom             = ethtypes.Bloom
+	Receipt           = ethtypes.Receipt
+	Receipts          = ethtypes.Receipts
+	ReceiptForStorage = ethtypes.ReceiptForStorage
+	LegacyTx          = ethtypes.LegacyTx
+	DynamicFeeTx      = ethtypes.DynamicFeeTx
+	BlobTx            = ethtypes.BlobTx
+	BlobTxSidecar     = ethtypes.BlobTxSidecar
+	Signer            = ethtypes.Signer
+	HomesteadSigner   = ethtypes.HomesteadSigner
+	FrontierSigner    = ethtypes.FrontierSigner
 
-	Transaction  = gethtypes.Transaction
-	Transactions = gethtypes.Transactions
-	TxByNonce    = gethtypes.TxByNonce
-	TxData       = gethtypes.TxData
+	Transaction  = ethtypes.Transaction
+	Transactions = ethtypes.Transactions
+	TxByNonce    = ethtypes.TxByNonce
+	TxData       = ethtypes.TxData
 )
 
 // The following constants are used directly as their upstream definitions.
 const (
-	BloomBitLength          = gethtypes.BloomBitLength
-	BloomByteLength         = gethtypes.BloomByteLength
-	ReceiptStatusFailed     = gethtypes.ReceiptStatusFailed
-	ReceiptStatusSuccessful = gethtypes.ReceiptStatusSuccessful
+	BloomBitLength          = ethtypes.BloomBitLength
+	BloomByteLength         = ethtypes.BloomByteLength
+	ReceiptStatusFailed     = ethtypes.ReceiptStatusFailed
+	ReceiptStatusSuccessful = ethtypes.ReceiptStatusSuccessful
 
 	// Transaction types.
-	LegacyTxType     = gethtypes.LegacyTxType
-	AccessListTxType = gethtypes.AccessListTxType
-	DynamicFeeTxType = gethtypes.DynamicFeeTxType
-	BlobTxType       = gethtypes.BlobTxType
+	LegacyTxType     = ethtypes.LegacyTxType
+	AccessListTxType = ethtypes.AccessListTxType
+	DynamicFeeTxType = ethtypes.DynamicFeeTxType
+	BlobTxType       = ethtypes.BlobTxType
 )
 
 // The following functions are used directly as their upstream definitions.
 var (
-	BloomLookup         = gethtypes.BloomLookup
-	BytesToBloom        = gethtypes.BytesToBloom
-	CreateBloom         = gethtypes.CreateBloom
-	NewReceipt          = gethtypes.NewReceipt
-	NewContractCreation = gethtypes.NewContractCreation
-	NewTransaction      = gethtypes.NewTransaction
+	BloomLookup         = ethtypes.BloomLookup
+	BytesToBloom        = ethtypes.BytesToBloom
+	CreateBloom         = ethtypes.CreateBloom
+	NewReceipt          = ethtypes.NewReceipt
+	NewContractCreation = ethtypes.NewContractCreation
+	NewTransaction      = ethtypes.NewTransaction
 
 	// Signers
-	NewEIP155Signer        = gethtypes.NewEIP155Signer
-	NewEIP2930Signer       = gethtypes.NewEIP2930Signer
-	NewLondonSigner        = gethtypes.NewLondonSigner
-	NewCancunSigner        = gethtypes.NewCancunSigner
-	MakeSigner             = gethtypes.MakeSigner
-	LatestSigner           = gethtypes.LatestSigner
-	LatestSignerForChainID = gethtypes.LatestSignerForChainID
-	SignTx                 = gethtypes.SignTx
-	SignNewTx              = gethtypes.SignNewTx
-	MustSignNewTx          = gethtypes.MustSignNewTx
-	Sender                 = gethtypes.Sender
+	NewEIP155Signer        = ethtypes.NewEIP155Signer
+	NewEIP2930Signer       = ethtypes.NewEIP2930Signer
+	NewLondonSigner        = ethtypes.NewLondonSigner
+	NewCancunSigner        = ethtypes.NewCancunSigner
+	MakeSigner             = ethtypes.MakeSigner
+	LatestSigner           = ethtypes.LatestSigner
+	LatestSignerForChainID = ethtypes.LatestSignerForChainID
+	SignTx                 = ethtypes.SignTx
+	SignNewTx              = ethtypes.SignNewTx
+	MustSignNewTx          = ethtypes.MustSignNewTx
+	Sender                 = ethtypes.Sender
 
 	// Transactions
-	NewTx        = gethtypes.NewTx
-	TxDifference = gethtypes.TxDifference
+	NewTx        = ethtypes.NewTx
+	TxDifference = ethtypes.TxDifference
 
 	// Errors
-	ErrTxTypeNotSupported = gethtypes.ErrTxTypeNotSupported
-	ErrGasFeeCapTooLow    = gethtypes.ErrGasFeeCapTooLow
+	ErrTxTypeNotSupported = ethtypes.ErrTxTypeNotSupported
+	ErrGasFeeCapTooLow    = ethtypes.ErrGasFeeCapTooLow
 )

core/types/log.go

@@ -26,11 +26,11 @@
 
 package types
 
-import gethtypes "github.com/ava-labs/libevm/core/types"
+import ethtypes "github.com/ava-labs/libevm/core/types"
 
 // Log represents a contract log event. These events are generated by the LOG opcode and
 // stored/indexed by the node.
-type Log = gethtypes.Log
+type Log = ethtypes.Log
 
 // FlattenLogs converts a nested array of logs to a single array of logs.
 func FlattenLogs(list [][]*Log) []*Log {

core/types/state_account.go

@@ -34,7 +34,7 @@ import (
 	"github.com/holiman/uint256"
 )
 
-//go:generate go run github.com/ethereum/go-ethereum/rlp/rlpgen -type StateAccount -out gen_account_rlp.go
+//go:generate go run github.com/ava-labs/libevm/rlp/rlpgen -type StateAccount -out gen_account_rlp.go
 
 // StateAccount is the Ethereum consensus representation of accounts.
 // These objects are stored in the main account trie.

eth/tracers/tracers.go

@@ -22,19 +22,19 @@ import (
 	"fmt"
 
 	"github.com/ava-labs/libevm/core/vm"
-	gethtracers "github.com/ava-labs/libevm/eth/tracers"
+	ethtracers "github.com/ava-labs/libevm/eth/tracers"
 )
 
 // Context contains some contextual infos for a transaction execution that is not
 // available from within the EVM object.
-type Context = gethtracers.Context
+type Context = ethtracers.Context
 
 // Tracer interface extends vm.EVMLogger and additionally
 // allows collecting the tracing result.
-type Tracer = gethtracers.Tracer
+type Tracer = ethtracers.Tracer
 
 // DefaultDirectory is the collection of tracers bundled by default.
-var DefaultDirectory = gethtracers.DefaultDirectory
+var DefaultDirectory = ethtracers.DefaultDirectory
 
 const (
 	memoryPadLimit = 1024 * 1024

params/config.go

@@ -32,7 +32,7 @@ import (
 	"github.com/ava-labs/avalanchego/upgrade"
 	"github.com/ava-labs/avalanchego/utils/constants"
 	"github.com/ava-labs/coreth/utils"
-	gethparams "github.com/ava-labs/libevm/params"
+	ethparams "github.com/ava-labs/libevm/params"
 )
 
 // Guarantees extras initialisation before a call to [ChainConfig.Rules].
@@ -216,11 +216,11 @@ var (
 // ChainConfig is stored in the database on a per block basis. This means
 // that any network, identified by its genesis block, can have its own
 // set of configuration options.
-type ChainConfig = gethparams.ChainConfig
+type ChainConfig = ethparams.ChainConfig
 
 // Rules wraps ChainConfig and is merely syntactic sugar or can be used for functions
 // that do not have or require information about the block.
 //
 // Rules is a one time interface meaning that it shouldn't be used in between transition
 // phases.
-type Rules = gethparams.Rules
+type Rules = ethparams.Rules

params/config_extra.go

@@ -14,7 +14,7 @@ import (
 	"github.com/ava-labs/coreth/predicate"
 	"github.com/ava-labs/coreth/utils"
 	"github.com/ava-labs/libevm/common"
-	gethparams "github.com/ava-labs/libevm/params"
+	ethparams "github.com/ava-labs/libevm/params"
 )
 
 const (
@@ -265,7 +265,7 @@ func configTimestampEqual(x, y *uint64) bool {
 
 // ConfigCompatError is raised if the locally-stored blockchain is initialised with a
 // ChainConfig that would alter the past.
-type ConfigCompatError = gethparams.ConfigCompatError
+type ConfigCompatError = ethparams.ConfigCompatError
 
 func newTimestampCompatError(what string, storedtime, newtime *uint64) *ConfigCompatError {
 	var rew *uint64

params/config_libevm.go

@@ -9,25 +9,25 @@ import (
 	"github.com/ava-labs/coreth/precompile/modules"
 	"github.com/ava-labs/coreth/precompile/precompileconfig"
 	"github.com/ava-labs/libevm/common"
-	gethparams "github.com/ava-labs/libevm/params"
+	ethparams "github.com/ava-labs/libevm/params"
 )
 
 // libevmInit would ideally be a regular init() function, but it MUST be run
 // before any calls to [ChainConfig.Rules]. See `config.go` for its call site.
 func libevmInit() any {
-	extras = gethparams.RegisterExtras(gethparams.Extras[*ChainConfigExtra, RulesExtra]{
+	extras = ethparams.RegisterExtras(ethparams.Extras[*ChainConfigExtra, RulesExtra]{
 		ReuseJSONRoot: true, // Reuse the root JSON input when unmarshalling the extra payload.
 		NewRules:      constructRulesExtra,
 	})
 	return nil
 }
 
-var extras gethparams.ExtraPayloads[*ChainConfigExtra, RulesExtra]
+var extras ethparams.ExtraPayloads[*ChainConfigExtra, RulesExtra]
 
 // constructRulesExtra acts as an adjunct to the [params.ChainConfig.Rules]
 // method. Its primary purpose is to construct the extra payload for the
 // [params.Rules] but it MAY also modify the [params.Rules].
-func constructRulesExtra(c *gethparams.ChainConfig, r *gethparams.Rules, cEx *ChainConfigExtra, blockNum *big.Int, isMerge bool, timestamp uint64) RulesExtra {
+func constructRulesExtra(c *ethparams.ChainConfig, r *ethparams.Rules, cEx *ChainConfigExtra, blockNum *big.Int, isMerge bool, timestamp uint64) RulesExtra {
 	var rules RulesExtra
 	if cEx == nil {
 		return rules