fix: only set address manager for OP Chain's ProxyAdmin

ethereum-optimism · Sep 25, 2024 · 8adedf0 · 8adedf0
1 parent 2d2c5e8
commit 8adedf0
Showing 1 changed file with 6 additions and 6 deletions.
diff --git a/packages/contracts-bedrock/scripts/deploy/Deploy.s.sol b/packages/contracts-bedrock/scripts/deploy/Deploy.s.sol
@@ -432,13 +432,13 @@ contract Deploy is Deployer {
         ProxyAdmin admin = new ProxyAdmin{ salt: _implSalt() }({ _owner: msg.sender });
         require(admin.owner() == msg.sender);
 
-        AddressManager addressManager = AddressManager(mustGetAddress("AddressManager"));
-        if (admin.addressManager() != addressManager) {
-            admin.setAddressManager(addressManager);
+        if (!_isSuperchain) {
+            AddressManager addressManager = AddressManager(mustGetAddress("AddressManager"));
+            if (admin.addressManager() != addressManager) {
+                admin.setAddressManager(addressManager);
+            }
+            require(admin.addressManager() == addressManager);
         }
-
-        require(admin.addressManager() == addressManager);
-
         save(proxyAdminName, address(admin));
         console.log("%s deployed at %s", proxyAdminName, address(admin));
         addr_ = address(admin);