search

smartcontracts / simple-optimism-node

The easiest way to run an Optimism node
MIT License
337 stars 115 forks source link

Update progress.sh #154

Closed anacrolix closed 4 months ago

anacrolix commented 4 months ago

Misc tweaks from banging my head while getting a node running locally.

Chomtana commented 4 months ago

So, this is mostly about syntax simplification, removing deprecated OP Goerli Testnet and typo fix right?

progress.sh main problem would be it can't work with snap sync

anacrolix commented 4 months ago

Yeah I had issues running it due to cast missing, added set -e to catch that and discovered that the expr math caused it to fail if I had progressed no blocks. So I kept poking it until it ended up in the current state.

But also I didn't know that about the snap sync thing, that explains why I was getting no progress.

It's a pretty minor thing to change but it's more robust and useful than it was and saves someone else tripping on everything I did and ending up at the same place.