Cleanup, comment out test code, prepare for release PR

Update signing to deal with NULL keystore

Allow NULL keystore to actually work

Remove debug outs

Check unserialized tx

Correct script serialization

Clip subscript end for cc properly

Update handling of relayed stake transactions

Change default note contents in Tx builder and cheat message change

Update stake cheat spend

Prepare for release keys

Add value to cheat catcher through SendChangeTo

Fix reference and test signature

Fix height or greater check

Cleanup cheatcatcher

Revert POS change

Cheatcatcher tests to remove and change to max sequential POS

Auto rescan if note witnesses are off

Auto-rescan wallets on reorg if note witnesses get out of sync

Adjust cheat catcher height

Keep cheatcatcher's lists in sync

Fix false positive error

Improve error reporting and wallet cleanup

Clean up POS checks

Timeout adjustment

Reduce lag timeouts

Remove spurios opret in tx builder, don't wait forever for blocks before mining, add delay to getblocktemplate

Clean up stake rebroadcasting and mempool checking

Error message only when cheatstaker parameter is invalid, not missing

Fix readme merge error

Zcash latest changes

Fix negative height issues for pruning cheat lists

update comments

Reorganized cheatcatcher for build issues

Build

Build

Cheatcatcher

Auto merge of #3587 - Eirik0:3586-fix-dump-comment, r=ebfull

Fix HDSeed comment in wallet dump

Closes #3586

Fix HDSeed comment

Simpler internal check

Auto merge of #3450 - leto:rpctest-readme, r=str4d

Fix some typos in rpc-tests readme

Debug comment cleanup

Update to Zcash rc1

Add staking control and reporting to getgenerate and setgenerate

Selectively reject Sapling transactions while synced below activation height. Add separate control of staking.

Cap the random additional stake of a block to 1/2 the current stake difficulty

Merge branch 'master' of https://github.com/zcash/zcash into dev

Sapling support for z_shieldcoinbase and more

Auto merge of #3578 - mdr0id:release-v2.0.1-rc1, r=bitcartel

Release v2.0.1-rc1

make-release.py: Updated release notes and changelog for 2.0.1-rc1.

make-release.py: Updated manpages for 2.0.1-rc1.

make-release.py: Versioning changes for 2.0.1-rc1.

Auto merge of #3517 - mdr0id:3388_persist_wallet, r=str4d

Sapling support for persisting wallet to disk

Closes #3388. Closes #3061.

Change name to Stake Guard

Update wallet_persistence test to verify spending notes after restart.

Fix error in display-only of winning stake POS hash when output prevout.n was not 0

Remove debugging output

Confirm positive and negative spends with and without priv keys

Add cc include dirs to make file

Update wallet_persistence test to verify wallet txs are persisted across restarts.

Complete nothing at stake solution, waiting for confirm of masks

Fix file permissions of QA test wallet_persistence.py

data parsing issues

Ignore decoding errors during -zapwallettxes

The undecoded wallet transaction is logged before proceeding, so later
recovery of metadata might be possible. But the fact that the user is
using -zapwallettxes is a clear indicator that they want
transactions removed from their wallet, so this is the priority.

Change behavior of ExtractDestination, Solver, and GetscriptAddress for crypto conditions

Additional null check

Add null check to deref

Fix deadlock from calling CWallet::AddSaplingIncomingViewingKey instead of CBasicKeyStore::AddSaplingIncomingViewingKey

Successful staking on Sapling wit Crypto Conditions

Auto merge of #3567 - bitcartel:3546_estimate_tx_size, r=bitcartel

For #3546. Improve estimated tx size for Sapling outputs.

move things around and some debugging code for odd issues

Auto merge of #3566 - bitcartel:3546_prohibit_sapling_map_orphans, r=bitcartel

For #3546. Shielded tx with missing inputs are not treated as orphans.

Persist Sapling payment address to IVK map

This ensures we remember any diversified addresses manually generated
outside the wallet.

Add gtest coverage of Sapling wallet persistence

Adding in rpc wallet sap for test_bitcoin

Serialize Sapling data in CWalletTx

If 2.0.0 nodes upgrade to 2.0.1 after Sapling has activated, the v4 Sapling
transactions in their wallet will be treated as corrupt, and a rescan will be
triggered which will overwrite the old-format transactions with the new
Sapling-aware format.

Persist Sapling key material in the wallet to disk

Store ExtFVK with encrypted Sapling spending key instead of FVK

This ensures that even when the wallet is encrypted, we can derive the default
Sapling payment address for our spending keys.

Merge pull request #26 from miketout/dev-zcash

Add stake consistency enforcement and near complete CoinbaseGuard val…

Add stake consistency enforcement and near complete CoinbaseGuard validation

Auto merge of #3559 - str4d:3552-testnet-min-difficulty-blocks, r=bitcartel

Allow minimum-difficulty blocks on testnet

This is a consensus rule change on testnet that will result in a chain split (leaving the stuck chain, as desired).

Reverts #2766 and part of #1338.
Closes #3552.

For #3546. Improve estimated tx size for Sapling outputs.

For #3546. Shielded tx with missing inputs are not treated as orphans.

Revert "wallet: Comment out HDSeed and CHDChain persistence to disk"

This reverts commit b7f9a7ae02ffcd4bce3df9e7c9e231b0d64ed0d4.

Auto merge of #3518 - str4d:3216-z_shieldcoinbase, r=str4d

Add Sapling support to z_shieldcoinbase

Part of #3216.

Fix pyflakes warnings

Merge pull request #25 from miketout/dev-zcash

Dev zcash

Latest Zcash updates and more CC for N@S

Comment debugging printf's - tracked down to Zcash raw tx bug, which means we should sync

Position PoW.MinDifficultyRules test after rule activates

Auto merge of #3561 - bitcartel:3560_update_sapling_note_data, r=bitcartel

Closes #3560. Update Sapling note data correctly when importing a key.

Explicitly check the min-difficulty flag against boost::none

It isn't clear how a boost::optional that holds 0 (which is the case for
regtest) is coerced to a boolean, unless you pore over the Boost
documentation. An explicit check is clearer.

Rename min-difficulty flag to remove off-by-one in the name

Update wallet_listreceived test for now-correct empty Sapling memos

Change error to qualified error vs. no error at all

Change block before header from error to warning

Auto merge of #3510 - arcalinea:3378_z_listunspent, r=bitcartel

Sapling support for z_listunspent

Closes #3378.

fix z_listunspent includeWatchonly logic

Only enable min-difficulty blocks on testnet from a particular height

The min-difficulty change is a bilateral consensus rule change, and so
must be conditionally enabled in order for the earlier section of the
chain to synchronise.

Technically this could be implemented as a network upgrade, but as this will
never be deployed to mainnet, a targeted fork will suffice.

Fix comparison