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

Disable Wallet Mode for Litecoin #78

Merged
merged 11 commits into from
Nov 23, 2013
Merged

Conversation

wtogami
Copy link
Member

@wtogami wtogami commented Oct 25, 2013

disablewallet=1 allows litecoind to operate with the BDB wallet disabled. This is known to reduce the memory necessary to operate a relay-only node by between 40-200MB. Pool servers and p2pool nodes can also operate with the disablewallet=1 by setting their payout address to an arbitrary address managed elsewhere. This can enhance security of mining nodes.

  • s/Wallet disabled.../Wallet disabled!/

@wtogami
Copy link
Member Author

wtogami commented Nov 2, 2013

The previous disablewallet branch was moved to exp-0.8.5.1-disablewalletrc4. This new branch is rebased from https://github.com/litecoin-project/bitcoinomg which is a cleaner backport than the previous branch. It includes a few other changes:

  • Bring back to the internal miner to bring us closer to upstream and make merges easier.
  • Add verifychain and getbestblockhash

This is based largely on commits already in Bitcoin master with the exception of bitcoin#2901 which is not net merged.

@wtogami
Copy link
Member Author

wtogami commented Nov 11, 2013

The previous disablewallet branch was moved to exp-0.8.5.1-disablewalletrc5. This new branch is rebased on master-0.8 with a one line merge fix for #84.

@wtogami
Copy link
Member Author

wtogami commented Nov 13, 2013

Backported GUI support for -disablewallet mode from bitcoin#3240

It was not a straightforward backport. It is possible that something in the GUI no longer works, so please test all features in the GUI with the wallet enabled.

laanwj and others added 11 commits November 14, 2013 00:44
This commit decouples the pMiningKey initialization and shutdown from the RPC
threads.

`getwork` and `getblocktemplate` rely on pMiningKey, and can also be ran
from the debug window in the UI even when the RPC server is not running.

Solves issue bitcoin#2706.

Conflicts:
	src/rpcmining.cpp
* Fix UpdateCoins() definition in main.h
* Remove pwalletMain reference from BitcoinMiner(), as it is passed
  a wallet argument.

Conflicts:
	src/main.h

Rebased-from: 1894684
rather than a key.

CreateNewBlockWithKey() helper is added to restore existing functionality,
making this an equivalent-transformation change.

Conflicts:
	src/miner.cpp
	src/miner.h

Rebased-from: fcc32b7b9ce5bc310cbad677da7e7bca3a01459a
getblocktemplate only uses certain portions of the coinbase transaction,
notably ignoring the coinbase TX output entirely.

Use CreateNewBlock() rather than CreateNewBlockWithKey(), eliminating
the needless key passing.

Should be zero behavior changes.
Conflicts:
	src/miner.cpp
	src/miner.h

Rebased-from: ad7d7cde5d30bcbfa7803c90daac7297b8a55e42
notably RPC.

Conflicts:
	src/init.cpp
	src/rpcmining.cpp
	src/rpcwallet.cpp
	src/bitcoinrpc.cpp

Rebased-from: 541a7fcd515a27ebaa3ae5f1a9a698dab3db6ead
Conflicts:
	src/rpcwallet.cpp

Rebased-from: b878f6a92436e3facd700e2e10c04f541cd6421b
Rebased-from: 8121047b20f6486bf049e642c3e6ca61682ffdd9
@wtogami
Copy link
Member Author

wtogami commented Nov 14, 2013

Rebased on master-0.8 with no changes. Five commits were added to master-0.8 as they are simple, well tested and simple prerequisites to make merging disablewallet cleaner. The remaining commits have equivalents in Bitcoin master and should be compared to the upstream commits to make sure merge errors didn't add things by accident.

Five commits added to master-0.8 were:

  • Revert "Litecoin: Remove native coin generation."
  • Litecoin: s/BitcoinMiner/LitecoinMiner/ and detect scrypt implementation
  • RPC: add getbestblockhash, to return tip of best chain
  • Pass check level, check depth to VerifyDB()
  • RPC: add 'verifychain', to verify chain database at runtime

@wtogami wtogami merged commit 99f3d18 into master-0.8 Nov 23, 2013
@wtogami
Copy link
Member Author

wtogami commented Nov 23, 2013

Merged

@wtogami wtogami deleted the exp-0.8.5.1-disablewallet branch September 2, 2014 03:13
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.

None yet

2 participants