Skip to content
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

ETC lite patch #2

Draft
wants to merge 144 commits into
base: master
Choose a base branch
from
Draft

ETC lite patch #2

wants to merge 144 commits into from

Conversation

meowsbits
Copy link
Owner

No description provided.

s1na and others added 27 commits September 13, 2022 23:03
* graphql: fix tx logs

* minor

* Use optimized search for selecting tx logs
* graphql: fix tx logs

* graphql: refactor test service setup

* graphql: add test for tx logs
Date: 2022-10-27 09:16:05-07:00
Signed-off-by: meows <[email protected]>
…p155 impls; ETC forkid impl

Date: 2022-10-27 09:46:38-07:00
Signed-off-by: meows <[email protected]>
…hash pkg

Date: 2022-10-27 11:09:17-07:00
Signed-off-by: meows <[email protected]>
…classic flag/config and associated ethash inits

Date: 2022-10-27 11:36:34-07:00
Signed-off-by: meows <[email protected]>
Date: 2022-10-27 11:37:02-07:00
Signed-off-by: meows <[email protected]>
…sis hashes

Date: 2022-10-27 11:46:48-07:00
Signed-off-by: meows <[email protected]>
…/tests-etc)

Date: 2022-10-27 13:00:03-07:00
Signed-off-by: meows <[email protected]>
Date: 2022-10-27 13:13:45-07:00
Signed-off-by: meows <[email protected]>
Date: 2022-10-27 13:17:15-07:00
Signed-off-by: meows <[email protected]>
A few tests remain failing; apparently
all for chain id reasons; do the tests
have proper chain id values?
They should be 61 for ETC.

Date: 2022-10-27 15:57:50-07:00
Signed-off-by: meows <[email protected]>
Date: 2022-10-27 16:07:14-07:00
Signed-off-by: meows <[email protected]>
Date: 2022-10-28 09:23:37-07:00
Signed-off-by: meows <[email protected]>
Date: 2022-11-01 10:29:42-07:00
Signed-off-by: meows <[email protected]>
Date: 2022-11-01 10:31:31-07:00
Signed-off-by: meows <[email protected]>
…1.12)

When running the 'evm' command via retesteth,
a panic occurs and the trace has this dep at the
top of the pile.

This is a blind attempt to resolve the issue
by upgrading dependencies.

Date: 2022-11-01 10:38:40-07:00
Signed-off-by: meows <[email protected]>
Running retesteth with --t8ntool --filltests --j8
(or any --jX > 1) caused the suite to panic.

Geth/evm was panicing about here.
This fixes it.

It (the panic) looked like this...

goroutine 8 [chan receive]:
github.com/ethereum/go-ethereum/core.(*txSenderCacher).cache(0x0?)
        github.com/ethereum/go-ethereum/core/sender_cacher.go:63 +0x3b
created by github.com/ethereum/go-ethereum/core.newTxSenderCacher
        github.com/ethereum/go-ethereum/core/sender_cacher.go:55 +0x75

goroutine 9 [chan receive]:
github.com/ethereum/go-ethereum/core.(*txSenderCacher).cache(0x0?)
        github.com/ethereum/go-ethereum/core/sender_cacher.go:63 +0x3b
created by github.com/ethereum/go-ethereum/core.newTxSenderCacher
        github.com/ethereum/go-ethereum/core/sender_cacher.go:55 +0x75

goroutine 10 [chan receive]:
github.com/ethereum/go-ethereum/core.(*txSenderCacher).cache(0x0?)
        github.com/ethereum/go-ethereum/core/sender_cacher.go:63 +0x3b
created by github.com/ethereum/go-ethereum/core.newTxSenderCacher
        github.com/ethereum/go-ethereum/core/sender_cacher.go:55 +0x75

goroutine 11 [chan receive]:
github.com/ethereum/go-ethereum/core.(*txSenderCacher).cache(0x0?)
        github.com/ethereum/go-ethereum/core/sender_cacher.go:63 +0x3b
created by github.com/ethereum/go-ethereum/core.newTxSenderCacher
        github.com/ethereum/go-ethereum/core/sender_cacher.go:55 +0x75

Date: 2022-11-01 08:16:04-07:00
Signed-off-by: meows <[email protected]>
Conflicts:
      cmd/utils/flags.go
      core/genesis.go
      eth/backend.go
      eth/ethconfig/config.go
      go.sum
      params/config.go
      params/version.go
Date: 2022-11-01 10:54:44-07:00
Signed-off-by: meows <[email protected]>
…date

Date: 2022-12-12 09:50:18-08:00
Signed-off-by: meows <[email protected]>
Date: 2022-12-12 14:00:56-08:00
Signed-off-by: meows <[email protected]>
The big idea here is to move the necessary
configuration changes for ETC to patterns
that will also support ETC tests and testnets.

This meant removing global ETC-only parameters
and changing them to field-based values on
the Config struct type.

Most (?) tests -- except, notably, EOF tests --
are passing.

Date: 2022-12-12 16:46:18-08:00
Signed-off-by: meows <[email protected]>
EOF1 is not implemented in this codebase yet.
This commit should be reverted once EOF1 support
is merged.

Date: 2022-12-12 16:50:54-08:00
Signed-off-by: meows <[email protected]>
Conflicts:
      consensus/ethash/ethash.go
@meowsbits meowsbits self-assigned this Dec 13, 2022
ralexstokes and others added 2 commits December 13, 2022 03:33
A comment suggests that contract creation happens if the recipient of a call is 0x00..00 ("zero address") but in fact the sender must be nil. The zero address is a regular valid address that is commonly used as a "burn" address.
* core, cmd: fill blockNumber in logs

* Update core/state/statedb.go

Co-authored-by: Martin Holst Swende <[email protected]>

* core/types: revert

* core/state: improve comments

Co-authored-by: Martin Holst Swende <[email protected]>
holiman and others added 30 commits January 31, 2023 10:16
* eth: cmd: deprecate personal namespace

* eth: cmd: move deprecation to node

* node: disable toml of enablepersonal

* node: disable personal on ipc as well

* Update node/node.go

Co-authored-by: Martin Holst Swende <[email protected]>

* console: error -> warn

* node: less roulette

---------

Co-authored-by: Martin Holst Swende <[email protected]>
accounts, build, mobile: remove Android and iOS support
This change improves reusability of the EVM struct. Two methods are added:

- SetBlockContext(...)
- SetTracer(...)

Other attributes like the TransactionContext and the StateDB can already be updated.
BlockContext and Tracer are partially not updateable right now. This change fixes it and
opens the potential to reuse an EVM struct in more ways.

Co-authored-by: Felix Lange <[email protected]>
…thereum#26232)

This change implements engine_getPayloadBodiesByHash and engine_getPayloadBodiesByRange, according to the specification at https://github.com/ethereum/execution-apis/blob/main/src/engine/shanghai.md#specification-4 .

Co-authored-by: Martin Holst Swende <[email protected]>
This change fixes a minor flaw in the check for ipc endpoint length. The max_path_size is the max path that an ipc endpoint can have, which is 208. However, that size concerns the null-terminated pathname, so we need to account for an extra null-character too.
…26607)

EIP-196 allows a zero value in the scalar argument to precompile `0x07`. This change adds a test for that case.
This PR moves some trie-related db accessor methods to a different file, and also removes the schema type. Instead of the schema type, a string is used to distinguish between hashbased/pathbased db accessors.
This also moves some code from trie package to rawdb package.

This PR is intended to be a no-functionality-change prep PR for ethereum#25963 .

---------

Co-authored-by: Gary Rong <[email protected]>
This PR moves core/beacon to beacon/engine so that beacon-chain related code has its own top level package which also can house the the beacon lightclient-code.
This updates the reference tests to the latest version.

Co-authored-by: Felix Lange <[email protected]>
This PR fixes this abi encoder/decoder to be more stringent.
It's the same as math.HexOrDecimal64, which has more uses across the codebase.
…hereum#26554)

According to the spec the payloadID needs to be random or dependent on all arguments, to prevent two payloads from clashing. This change adds withdrawals into the payload derivation. 

---------

Co-authored-by: [email protected] <[email protected]>
Co-authored-by: Martin Holst Swende <[email protected]>
Co-authored-by: Felix Lange <[email protected]>
* log: allow tabs in log messages

This fixes a regression where panic reports in RPC handlers were quoted
because they contain tab characters.

* Update format.go
…6637)

This change ports some changes from the main PBSS PR:

  - get rid of callback function in `trie.Database.Commit` which is not required anymore
  - rework the `nodeResolver` in `trie.Iterator` to make it compatible with multiple state scheme
  - some other shallow changes in tests and typo-fixes
…26624)


Co-authored-by: Martin Holst Swende <[email protected]>
Co-authored-by: Marius van der Wijden <[email protected]>
Conflicts:
      cmd/utils/flags.go
      core/forkid/forkid.go
      core/vm/interpreter.go
      eth/catalyst/api_test.go
      go.mod
      go.sum
      params/config.go
Date: 2023-02-08 09:22:10-08:00
Signed-off-by: meows <[email protected]>
Date: 2023-02-08 09:34:42-08:00
Signed-off-by: meows <[email protected]>
In the etc-tests accessList tests exist with
name 'Merged'.
Should be 'Merge'.

Date: 2023-02-08 11:47:45-08:00
Signed-off-by: meows <[email protected]>
…1.20+)

Date: 2023-02-08 15:04:14-08:00
Signed-off-by: meows <[email protected]>
Date: 2023-02-08 15:04:55-08:00
Signed-off-by: meows <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.