-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Document the Finalized Initcode #918
Comments
This is probably the problem I was having. It is still a good idea to commit the initcode. |
For |
constructor arguments are part of the initcode so they need to be taken into account. The hash that you found ( |
Component
No response
Describe the suggested feature and problem it solves.
The initcode hash is published but the initcode is not. If the initcode matching the initcode hash were to be lost, it could cause issues with the final deployment next year.
Describe the desired implementation.
Commit the finalized output files into version control. These include the initcode and the ABI.
Describe alternatives.
Commit separate files with the final initcode and ABI.
Additional context.
I am having trouble generating initcode that matches the published initcode hash.
I worry that the initcode hash in the contest contract may be incorrect because I cannot reproduce it. I calculate
0x44d2e366954aa5a7f0605ffd66ca57a5a71987845939d8d8c79c716d8dfbb3de
instead. Here is how:I checkout
main
and doforge update
andforge build
.I do
cat out/PoolManager.t.sol/PoolManagerTest.json | jq -r .bytecode.object > v4.out
In python3 I do:
The result is
0x44d2e366954aa5a7f0605ffd66ca57a5a71987845939d8d8c79c716d8dfbb3de
.The text was updated successfully, but these errors were encountered: