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

core: fixes for Prague fork in GenerateChain #30924

Merged
merged 2 commits into from
Dec 17, 2024

Conversation

fjl
Copy link
Contributor

@fjl fjl commented Dec 17, 2024

Adding some missing functionality I noticed while updating the hivechain tool for the Prague fork:

  • we forgot to process the parent block hash
  • added ConsensusLayerRequests to get the requests list of the block

@fjl
Copy link
Contributor Author

fjl commented Dec 17, 2024

It took me quite a while to get this working because of a quirk in GenerateChain: when the added block of code executes, header difficulty will always be set to a non-zero value, which makes the EVM assume the merge hasn't happened yet. This ultimately leads to the system call failing because it attempts to use PUSH0, which was added after the merge.

I need this to collect the requests in cmd/hivechain.
@fjl fjl changed the title core: add EIP-2935 support in GenerateChain core: fixes for Prague in GenerateChain Dec 17, 2024
Copy link
Contributor

@holiman holiman left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@fjl fjl changed the title core: fixes for Prague in GenerateChain core: fixes for Prague fork in GenerateChain Dec 17, 2024
@fjl fjl added this to the 1.14.13 milestone Dec 17, 2024
@fjl fjl merged commit 71c28d8 into ethereum:master Dec 17, 2024
3 checks passed
GrapeBaBa pushed a commit to optimism-java/shisui that referenced this pull request Dec 23, 2024
Adding some missing functionality I noticed while updating the hivechain
tool for the Prague fork:

- we forgot to process the parent block hash
- added `ConsensusLayerRequests` to get the requests list of the block
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.

2 participants