ethereum / ethereumj

DEPRECATED! Java implementation of the Ethereum yellowpaper. For JSON-RPC and other client features check Ethereum Harmony
GNU Lesser General Public License v3.0
2.18k stars 1.1k forks source link

Fix/ropsten constantinople #1214

Closed zilm13 closed 5 years ago

zilm13 commented 5 years ago

To be updated.

Not yet ready for merge.

Fixed: Constantinople fork has not started at correct block. Finished by updating ropsten.json and improving net config logging to reduce mistakes in the future:

16:30:14.153 DEBUG [general]  Rendering net config from genesis org.ethereum.core.genesis.GenesisConfig@57c03d88 ...
16:30:14.153 DEBUG [general]  Block #0 => Frontier
16:30:14.154 DEBUG [general]  Block #0 => Homestead
16:30:14.156 DEBUG [general]  Block #10 => EIP158, chainId: 3
16:30:14.156 DEBUG [general]  Block #1700000 => Byzantium, chainId: 3
16:30:14.157 DEBUG [general]  Block #4230000 => Constantinople, chainId: 3
16:30:14.157 DEBUG [general]  Finished rendering net config from genesis org.ethereum.core.genesis.GenesisConfig@57c03d88
16:30:14.158 INFO [general]  Blockchain config BaseNetConfig{blockNumbers= #0 => HomesteadConfig, #10 => org.ethereum.config.net.JsonNetConfig$1@518caac3, #1700000 => org.ethereum.config.net.JsonNetConfig
$2@68034211, #4230000 => org.ethereum.config.net.JsonNetConfig$3@4f74980d (total: 4)}
zilm13 commented 5 years ago

@mkalinin ready for review