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

Feature request: --disable-metadata as CLI option to forge build #7488

Closed
PetarMax opened this issue Mar 25, 2024 · 3 comments · Fixed by #7684
Closed

Feature request: --disable-metadata as CLI option to forge build #7488

PetarMax opened this issue Mar 25, 2024 · 3 comments · Fixed by #7684
Assignees
Labels
T-feature Type: feature

Comments

@PetarMax
Copy link

Component

Forge

Describe the feature you would like

An equivalent of passing --no-cbor-metadata --metadata-hash none to solc, or adding

bytecode_hash = "none"
cbor_metadata = false

to foundry.toml, this CLI feature would help stabilise the generated EVM bytecode.

Additional context

I am working on the development of Kontrol, a tool for formal verification of Ethereum smart contracts. As part of our verification pipeline, we would like to be able to switch this metadata manually using CLI. This would allow our analyses to scale better, while not requiring client-side changes.

@PetarMax PetarMax added the T-feature Type: feature label Mar 25, 2024
@PetarMax PetarMax changed the title --disable-metadata as CLI option to forge build Feature request: --disable-metadata as CLI option to forge build Mar 25, 2024
@zerosnacks zerosnacks self-assigned this Mar 29, 2024
@ehildenb
Copy link

@zerosnacks is there any progress on this? This would seriously improve the ease with which formal verification tools can re-use proof executions between runs.

@zerosnacks
Copy link
Member

zerosnacks commented Apr 16, 2024

Hi @ehildenb, thanks for the ping - will be picking this up now, aiming for a reviewable draft before EOD.

@zerosnacks
Copy link
Member

@ehildenb & @PetarMax

Added as --no-metadata in #7684

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
T-feature Type: feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants