-
Notifications
You must be signed in to change notification settings - Fork 5.4k
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
Add meta EIP for Metropolis/Byzantium #609
Conversation
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.
Looks good to me except for the activation blocks (which hasn't been decided yet anyway).
EIPS/eip-draft_hfmeta_metropolis.md
Outdated
- EIP 86 (Abstraction of transaction origin and signature) | ||
- EIP 96 (Blockhash refactoring) | ||
- EIP 98 (Removal of intermediate state roots from receipts) | ||
- EIP 100 (Change difficulty adjustment to target mean block time including uncles) |
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.
EIP 100 has different options (Specification and Specification (1b)). So one of these needs to be specified.
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.
@Souptacular @vbuterin do you know which one (if any) is currently "accepted"?
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.
I'm only seeing (1b) implemented.
EIPS/eip-draft_hfmeta_metropolis.md
Outdated
- Included EIPs: | ||
- EIP 86 (Abstraction of transaction origin and signature) | ||
- EIP 96 (Blockhash refactoring) | ||
- EIP 98 (Removal of intermediate state roots from receipts) |
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.
EIP 98 also has different options. In the last coredev meeting, we chose option 1.
EIPS/eip-draft_hfmeta_metropolis.md
Outdated
- EIP 198 (Precompiled contract for bigint modular exponentiation) | ||
- EIP 211 (New opcodes: RETURNDATASIZE and RETURNDATACOPY) | ||
- EIP 214 (New opcode STATICCALL) | ||
|
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.
EIP 658 now needs to be added.
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.
This looks done.
EIPS/eip-draft_hfmeta_metropolis.md
Outdated
Category: Core | ||
Status: Draft | ||
Created: 23/04/2017 | ||
Requires: 86, 96, 98, 100, 140, 196, 197, 198, 211, 214 |
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.
658 should be added and 86 should be moved to the next hard fork.
Given the split of Metropolis into two, we should create a second issue. |
EIPS/eip-draft_hfmeta_metropolis.md
Outdated
## Preamble | ||
|
||
EIP: <to be assigned> | ||
Title: Hardfork Meta: Metropolis |
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.
New official name is Byzantium
.
EIPS/eip-draft_hfmeta_metropolis.md
Outdated
Type: Standard Track | ||
Category: Core | ||
Status: Draft | ||
Created: 23/04/2017 |
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.
Date needs to change to standard format.
No description provided.