-
Notifications
You must be signed in to change notification settings - Fork 725
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
Grayglacier HF Support (master) #1983
Changes from 3 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -82,6 +82,11 @@ | |
"block": 13773000, | ||
"forkHash": "0x20c327fc" | ||
}, | ||
{ | ||
"name": "grayGlacier", | ||
"block": 15050000, | ||
"forkHash": null | ||
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. Please calculate an explicit forkHash here, you can do this with the respective method in the 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. Ok. the grayGlacier markdown file had forkHash: TBA -- i'll check with other clients |
||
}, | ||
{ | ||
"name": "mergeForkIdTransition", | ||
"block": null, | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
{ | ||
"name": "EIP-5133", | ||
"number": 5133, | ||
"comment": "Delaying Difficulty Bomb to mid-September 2022", | ||
"url": "https://eips.ethereum.org/EIPS/eip-5133", | ||
"status": "Draft", | ||
"minimumHardfork": "grayGlacier", | ||
"gasConfig": {}, | ||
"gasPrices": {}, | ||
"vm": {}, | ||
"pow": { | ||
"difficultyBombDelay": { | ||
"v": 11400000, | ||
"d": "the amount of blocks to delay the difficulty bomb with" | ||
} | ||
} | ||
} |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -21,6 +21,7 @@ export enum Hardfork { | |
Berlin = 'berlin', | ||
London = 'london', | ||
ArrowGlacier = 'arrowGlacier', | ||
GrayGlacier = 'grayGlacier', | ||
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. Unrelated, but since you are already working on the code: can you please move (general note: please do all change requests on both PRs, maybe do all in one commit and then cherry-pick over?) |
||
Shanghai = 'shanghai', | ||
MergeForkIdTransition = 'mergeForkIdTransition', | ||
Merge = 'merge', | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,11 @@ | ||
{ | ||
"name": "grayGlacier", | ||
"comment": "Delaying the difficulty bomb to Mid September 2022", | ||
"url": "https://github.com/ethereum/execution-specs/blob/master/network-upgrades/mainnet-upgrades/gray-glacier.md", | ||
"status": "Draft", | ||
"eips": [5133], | ||
"gasConfig": {}, | ||
"gasPrices": {}, | ||
"vm": {}, | ||
"pow": {} | ||
} |
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.
So to your question with the tests: just copy the whole file over and move to the local Block test data folder and reference through the local path. You can maybe add a TODO here that this should be replaced with the
ethereum-tests
folder again at some point.