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

Add pre-installs for deployment factories #30

Merged
merged 12 commits into from
Jan 15, 2025
Merged

Conversation

rmeissner
Copy link
Contributor

This RIP proposes a set of known deployment factories that should be pre-installed on every rollup to make it easier to deploy contracts in a trustless, deterministic way. In the best case this also allows for same address deployments of the same contracts across multiple chains.

rmeissner and others added 9 commits July 5, 2024 11:30
@CarlBeek
Copy link
Contributor

Hey @rmeissner I have assigned the next available number, please update to RIP-7740

RIPS/rip-7740.md Outdated
title: Preinstall deterministic deployment factories
description: Proposal to add deployment factory contracts at common addresses to enable deterministic contract deployments
author: Richard Meissner (@rmeissner), Mikhail Mikheev (@mmv08), Nicholas Rodrigues Lordello (@nlordell)
discussions-to: https://ethereum-magicians.org/t/eip-proposal-create2-contract-factory-precompile-for-deployment-at-consistent-addresses-across-networks/6083/29
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggest creating a new topic on Eth Magicians to discuss this RIP

@Agusx1211
Copy link

Commenting here as the magicians thread is kinda off topic.
It may be worth it to include these two factories too:
https://eips.ethereum.org/EIPS/eip-2470
https://gist.github.com/Agusx1211/de05dabf918d448d315aa018e2572031

@oneski
Copy link

oneski commented Aug 22, 2024

any consideration to adding multicall to this list?

Copy link
Collaborator

@nconsigny nconsigny left a comment

Choose a reason for hiding this comment

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

Should change to RRC before merging

RIPS/rip-7740.md Outdated Show resolved Hide resolved
@nconsigny nconsigny marked this pull request as ready for review January 14, 2025 01:04
@rmeissner
Copy link
Contributor Author

@oneski @Agusx1211 I added a new discussion thread to align on the open questions

Copy link
Collaborator

@nconsigny nconsigny left a comment

Choose a reason for hiding this comment

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

LGTM

@nconsigny nconsigny merged commit a0fbf3f into ethereum:master Jan 15, 2025
1 check passed
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.

7 participants