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

settable etherbase #580

Merged
merged 1 commit into from
Mar 27, 2015
Merged

Conversation

zelig
Copy link
Contributor

@zelig zelig commented Mar 26, 2015

  • etherbase flag for block reward destination
  • coinbase => etherbase
  • CLI- eth Config -> eth, xeth -> RPC / Miner
  • use primary instead of coinbase as the unlock magic wildcard
  • accounts: firstAddr/Coinbase -> Primary

cont'd from #550 #541
https://github.com/ethereum/go-ethereum/wiki/Managing-your-accounts
system testing: ethereum/system-testing#61

- etherbase flag for block reward destination
- coinbase => etherbase
- CLI- eth Config -> eth, xeth -> RPC / Miner
- use primary instead of coinbase as the unlock magic wildcard
- accounts: firstAddr/Coinbase -> Primary
@zelig zelig added the feature label Mar 26, 2015
@zelig zelig self-assigned this Mar 26, 2015
@zelig zelig added this to the Frontier milestone Mar 26, 2015
@obscuren obscuren mentioned this pull request Mar 26, 2015
4 tasks
obscuren added a commit that referenced this pull request Mar 27, 2015
@obscuren obscuren merged commit 0b4b2b8 into ethereum:develop Mar 27, 2015
@zelig zelig deleted the frontier/cli-key branch December 8, 2015 20:51
ngtuna added a commit to ngtuna/tomochain that referenced this pull request Jul 22, 2019
tony-ricciardi pushed a commit to tony-ricciardi/go-ethereum that referenced this pull request Jan 20, 2022
…thereum#580)


- Reinserts previous lock to ensure changes are atomic.
- Do RefreshValPeers within val_enode_table
- Add ContainsByAddress() to valSet
- Moves logic to ValidatorEnodesHandler (instead of ValEnodeDB)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants