Skip to content

core, cmd, internal: rework BAL json marshalling to adhere EELS - #34972

Merged
jwasinger merged 1 commit into
ethereum:masterfrom
rjl493456442:eels-bal
May 20, 2026
Merged

core, cmd, internal: rework BAL json marshalling to adhere EELS#34972
jwasinger merged 1 commit into
ethereum:masterfrom
rjl493456442:eels-bal

Conversation

@rjl493456442

Copy link
Copy Markdown
Member

It's a change to BAL json marshalling and t8n tooling to adhere the EELS definition.

Comment thread cmd/evm/internal/t8ntool/execution.go
@jwasinger
jwasinger merged commit 918d46b into ethereum:master May 20, 2026
9 checks passed
@jsign

jsign commented May 22, 2026

Copy link
Copy Markdown
Contributor

@rjl493456442, I think the blockAccessList correct type for EELS is Bytes (RLP encoded), instead of the structured type added here?

@jwasinger

Copy link
Copy Markdown
Contributor

Yes. It's fixed in bal-devnet-7 but hasn't been applied to master yet.

@jsign

jsign commented May 22, 2026

Copy link
Copy Markdown
Contributor

Yes. It's fixed in bal-devnet-7 but hasn't been applied to master yet.

@jwasinger, I think what you linked there is ExecutableData which is EngineAPI related. Here I'm referring to t8n.

@jsign

jsign commented May 25, 2026

Copy link
Copy Markdown
Contributor

OK, I think now has been properly fixed in #35025

0xjvn pushed a commit to 0xjvn/go-ethereum that referenced this pull request Jun 5, 2026
…reum#34972)

It's a change to BAL json marshalling and t8n tooling to adhere the EELS
definition.
AdityaSripal pushed a commit to AdityaSripal/go-ethereum that referenced this pull request Jun 21, 2026
…reum#34972)

It's a change to BAL json marshalling and t8n tooling to adhere the EELS
definition.
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.

3 participants