Skip to content

Conversation

karalabe
Copy link
Member

@karalabe karalabe commented Jul 7, 2016

This PR implements setting the --support-dao-fork and --oppose-dao-fork flags. As of now they only modify a single database entry specifying whether the current default behavior should change.

  • If no dao fork number is set, --support-dao-fork will set it to either mainnet/testnet number
  • If a dao fork number is set, --support-dao-fork will leave it at its present value
  • If a dao fork number is set, --oppose-dao-fork will remove it

Caveat: setting the dao number from a custom genesis file, removing it via --oppose-dao-fork and reenabling it via --support-dao-fork will result in the mainnet/testnet number being set, the original private number being lost. Don't use this willy nilly on private networks; rather use geth init genesis.json for those scenarios.

This PR further drops support to the deprecated --genesis flag. This is important because the --support-dao-fork and --oppose-dao-fork flags need to know the previous state/number in the database to ensure it's not accidentally overwritten! We do not know this with the --genesis flag as the user might supply something we can only interpret much later in the code. Note, for main and test network we know in advance what their genesis file would contain, so that's not an issue.

@robotally
Copy link

robotally commented Jul 7, 2016

Vote Count Reviewers
👍 0
👎 0

Updated: Sat Jul 16 10:10:08 UTC 2016

@karalabe
Copy link
Member Author

Superseeded and merged in #2814.

@karalabe karalabe closed this Jul 16, 2016
@obscuren obscuren removed the review label Jul 16, 2016
sduchesneau pushed a commit to streamingfast/go-ethereum that referenced this pull request Feb 10, 2025
…thereum#2788)

Bumps [github.com/golang-jwt/jwt/v4](https://github.com/golang-jwt/jwt) from 4.5.0 to 4.5.1.
- [Release notes](https://github.com/golang-jwt/jwt/releases)
- [Changelog](https://github.com/golang-jwt/jwt/blob/main/VERSION_HISTORY.md)
- [Commits](golang-jwt/jwt@v4.5.0...v4.5.1)

---
updated-dependencies:
- dependency-name: github.com/golang-jwt/jwt/v4
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
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.

3 participants