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

Update to go-ethereum 1.12.0 #199

Merged
merged 8 commits into from
Aug 11, 2023
Merged

Update to go-ethereum 1.12.0 #199

merged 8 commits into from
Aug 11, 2023

Conversation

Xenomega
Copy link
Member

@Xenomega Xenomega commented Aug 9, 2023

This PR brings support for go-ethereum 1.12.0, courtesy of @anishnaik updating medusa-geth here and making the initial updates to medusa on this branch.

I've reviewed Anish's updates and made a rather minor fix for a minor chain revert issue that was caused by an earlier PR. After verifying the following, we can merge:

  • Support for the new Solidity version (which requires go-ethereum 1.12.0) is confirmed to work.
  • Verify SourceMap does not encounter problems with introduction of the PUSH0 instruction, as PUSH instructions must be specially considered when separating opcodes from operands. This just involves testing report generation with PUSH0 instructions.

@Xenomega Xenomega merged commit cd8605e into master Aug 11, 2023
8 checks passed
@Xenomega Xenomega deleted the dev/geth-1.12.0 branch August 11, 2023 15:28
damilolaedwards pushed a commit that referenced this pull request Nov 16, 2023
* Updated CallMessage to be compliant with core.Message made in new go-ethereum

---------

Co-authored-by: anishnaik <anish.r.naik@gmail.com>
s4nsec pushed a commit that referenced this pull request Jul 9, 2024
* Updated CallMessage to be compliant with core.Message made in new go-ethereum

---------

Co-authored-by: anishnaik <anish.r.naik@gmail.com>
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