Skip to content
This repository was archived by the owner on Nov 6, 2020. It is now read-only.

[spec] fix rinkeby spec#11108

Merged
niklasad1 merged 1 commit into
masterfrom
ao-rinkeby-fix
Oct 2, 2019
Merged

[spec] fix rinkeby spec#11108
niklasad1 merged 1 commit into
masterfrom
ao-rinkeby-fix

Conversation

@ordian
Copy link
Copy Markdown
Member

@ordian ordian commented Oct 1, 2019

This caused genesis hash mismatch.

@ordian ordian added A0-pleasereview 🤓 Pull request needs code review. A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). B1-patch-beta 🕷🕷 M2-config 📂 Chain specifications and node configurations. B0-patch-stable 🕷 Pull request should also be back-ported to the stable branch. labels Oct 1, 2019
@ordian ordian added this to the 2.7 milestone Oct 1, 2019
Copy link
Copy Markdown
Collaborator

@niklasad1 niklasad1 left a comment

Choose a reason for hiding this comment

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

damn, mybad

@niklasad1 niklasad1 added A8-looksgood 🦄 Pull request is reviewed well. and removed A0-pleasereview 🤓 Pull request needs code review. labels Oct 1, 2019
@dvdplm
Copy link
Copy Markdown
Collaborator

dvdplm commented Oct 1, 2019

Ah... while I was working on the json-tests I actually noticed that we had this odd 1wei balance for the builtins. Apparently this is the workaround we used years back when we had a consensus issue with geth and the ripemd bultin contract was removed due to it being considered empty. Sending 1wei to the builtins made them non-empty and so they were not removed...

@dvdplm dvdplm added A7-looksgoodtestsfail 🤖 Pull request is reviewed well, but cannot be merged due to tests failing. and removed A8-looksgood 🦄 Pull request is reviewed well. labels Oct 1, 2019
@niklasad1 niklasad1 added A8-looksgood 🦄 Pull request is reviewed well. and removed A7-looksgoodtestsfail 🤖 Pull request is reviewed well, but cannot be merged due to tests failing. labels Oct 2, 2019
@niklasad1 niklasad1 merged commit d243b15 into master Oct 2, 2019
@niklasad1 niklasad1 deleted the ao-rinkeby-fix branch October 2, 2019 08:52
niklasad1 pushed a commit that referenced this pull request Oct 2, 2019
niklasad1 pushed a commit that referenced this pull request Oct 2, 2019
This was referenced Oct 2, 2019
ordian pushed a commit that referenced this pull request Oct 2, 2019
* [spec] fix rinkeby spec (#11108)

* use images from our registry (#11105)
ordian pushed a commit that referenced this pull request Oct 2, 2019
* [spec] fix rinkeby spec (#11108)

* use images from our registry (#11105)
@ordian ordian mentioned this pull request May 12, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A2-insubstantial 👶 Pull request requires no code review (e.g., a sub-repository hash update). A8-looksgood 🦄 Pull request is reviewed well. B0-patch-stable 🕷 Pull request should also be back-ported to the stable branch. M2-config 📂 Chain specifications and node configurations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants