-
Notifications
You must be signed in to change notification settings - Fork 8
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
ECIP-1054: Support for ETH Byzantium EVM and Protocol Upgrades #10
Conversation
Along w/ a few other minor grammatical tweaks
Fixes EIP658 name; Tx status code -> receipt status code Co-Authored-By: meowsbits <[email protected]>
Prioritize the actual outcomes of the proposed features, not first the case of interoperability. Rel #2 (comment)
Fixes typo. Co-Authored-By: meowsbits <[email protected]>
Makes sentence more assertive. Co-Authored-By: meowsbits <[email protected]>
This comment has been minimized.
This comment has been minimized.
|
||
- Multi Geth: all Byzantium features | ||
- Parity Ethereum: all Byzantium features | ||
- Geth Classic: partial support for EIP-658 receipt status code change |
This comment was marked as resolved.
This comment was marked as resolved.
Sorry, something went wrong.
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 reflects the current state, not the desired state. In general, this section is used to point out a reference implementation which can be used by other client developer teams.
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.
IMO this should just be removed then; a spec should not be a roster of supposed current implementations unless the implementation(s) is part of the spec.
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 is exactly what I want to see happen.
- short
- to the point
- specifies All-client -support directly in the proposal
- Its ambitious, yet has a specific and realistic timeline
LGTM. Let's begin the discussion with other dev teams
Enable the outstanding Ethereum Foundation _Spurious Dragon_ and _Byzantium_ network protocol upgrades Isn't Spurious Dragon also in ETC at the moment? |
No, EIP-161 and 170 are missing which could cause complications and incompatibilities. After discussing with @sorpaas I decided it makes sense to include them in Atlantis. |
@soc1c Can you say a little more about that discussion (re/ EIP161,170) -- just for FYI for readers and to document the reasoning? |
IMO t-minus 6 months from now is a rather long time... and I'm usually the skeptical one trying to push these dates back for safety 😹 ... These changes are really important for ETC and it's current and potential developers; and for the sake of the chain's relevance to ETH-facing contract developers, they should happen ASAP. As in the doc, Parity-Ethereum, Mantis, and Multi-Geth are ready for liftoff essentially now.
(My block time calculations for reference, where
|
Six months is the minimum possible time to prepare for the hardfork. On the testnets it's already in four months and Geth Classic (any of them) does not even have it implemented yet. We don't have the developer power that Ethereum Foundation has and the ETH network uses a nine months hardfork cycle. So to be honest with you, six months was the most optimistic timeline I could propose. But happy to hear other opinions. However, I am highly sceptical that we can fork earlier than I proposed. |
EIP-{161/170} are proposed to enable maximum Byzantium compatibility and to avoid unwanted surprises in the behaviour of any implemented Byzantium proposal. Also, as @sorpaas pointed out, this will simplify testing massively. |
Any further comments @etclabscore/developers ? |
Alright, that's OK with me. Better safe. Do we have any points of contact for maintainers for either of the Geth Classics who would be willing to a) commit to the work, and b) give a timeframe on that work? Or maybe, better, the ECIP should not specify or even mention clients at all; that's the job of metadata around the ECIP organizing and encouraging client participation; there could, after all, be any number of "unofficial" clients that can't be counted on or accounted for, and a client list is not a true "specification," it's an informal team roster at best. |
I'm all for coordinating as much as humanly possible. Something like the joint call last week, but to discuss the upgrade. |
Again, please don't confuse governance with writing a spec. I want feedback here and now whether this proposal makes sense and is sound from a technical perspective. Things that are not relevant right now: block numbers, client implementation status, etc. That's something we track in the proposal, but it's not part of the technical specification. So if there is nothing else, I would like to move on. :) |
EIP-161 seems to introduce quite a bit of tech debt and does not significantly affect ETH compatibility:
(note that if we dont include 161, it will affect EIP-1052 but it should be a trivial fix ) |
An alternative to EIP-161 is to split this into two (hard fork phases):
The two phases are needed because (1) locks in which empty accounts have been saved as (2) would require a priori knowledge that is very unlikely to know in absolute. |
Atlantis finalization call ethereumclassic/ECIPs#78 |
Atlantis Hardfork Meta
Enable the outstanding Ethereum Foundation Spurious Dragon and Byzantium network protocol upgrades on the Ethereum Classic network in a hard-fork code-named Atlantis to enable maximum compatibility across these networks.