Simplify PartitionCheck

Temporarily disable test

Make NetworkUpgradeAvailable a method of Params

Update block subsidy halving for zip208

Update test_pow for shorter block times

Update pow_tests for shorter block times

Shorter block times rpc test

Make nextHeight required in CalculateNextWorkRequired

Modify PartitionCheck to be aware of pre & post Blossom target spacing.

Replace nPoWTargetSpacing -> PoWTargetSpacing() in tests

Replace nPoWTargetSpacing -> PoWTargetSpacing() in ProcessMessage()

Update GetBlockTimeout() to take height parameter.

Update PoW function calls to pass in height.

Replace nPoWTargetSpacing -> PoWTargetSpacing()

Remove use of redundant member nPowTargetSpacing.

Redefine PoW functions to accept height parameter.

Introduce target spacing constants and redefine struct member
variable nPoWTargetSpacing as a member function.  The height
parameter is used to determine if Blossom has activated yet.

Auto merge of #4054 - rex4539:haveshieldedrequirements, r=Eirik0

Rename joinsplit to shielded

Closes https://github.com/zcash/zcash/issues/3550

Auto merge of #4055 - rex4539:rename-findwallettx, r=Eirik0

Rename FindWalletTx to FindWalletTxToZap

Closes https://github.com/zcash/zcash/issues/3572

Auto merge of #4052 - rex4539:rename-vjoinsplit, r=Eirik0

Rename vjoinsplit to vJoinSplit

Closes https://github.com/zcash/zcash/issues/3188

Auto merge of #4058 - daira:release-v2.0.6, r=daira

Release v2.0.6

Test plan (ECC internal): https://docs.google.com/document/d/1EVDxdFRgQR5xBzA2wRsw02sb-yF2h26FyVnSjqfpayg/edit#

make-release.py: Updated release notes and changelog for 2.0.6.

make-release.py: Updated manpages for 2.0.6.

make-release.py: Versioning changes for 2.0.6.

Add coding declaration to zcutil/release-notes.py

Signed-off-by: Daira Hopwood <[email protected]>

Auto merge of #4057 - Eirik0:2.0.6-release-notes, r=daira

Notable changes for 2.0.6

Tweak author aliases.

Signed-off-by: Daira Hopwood <[email protected]>

Notable changes for 2.0.6

Co-authored-by: Daira Hopwood <[email protected]>

Rename FindWalletTx to FindWalletTxToZap

Rename joinsplit to shielded

Rename vjoinsplit to vJoinSplit

Auto merge of #4049 - mms710:issue_3504_fix, r=Eirik0

Fixes issue #3504

Changes the command to obtain the GCC version in the GitHub issue template to `gcc --version`, and adds a couple other useful commands.

ld --version doesn't work on macOS.

Signed-off-by: Daira Hopwood <[email protected]>

Fixes issue #3504: Changes to --version and adds a couple other useful commands.

Auto merge of #4048 - charlieok:add_stretch_suite_to_gitian_descriptors, r=mdr0id

Add stretch to list of suites in gitian linux descriptors

We want to do reproducible builds for Debian stretch (9) in addition to Debian jessie (8).

Add stretch to list of suites in gitian linux descriptors

Auto merge of #4046 - daira:release-v2.0.6-rc1, r=mdr0id

Release v2.0.6-rc1

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

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

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

Auto merge of #3863 - LarryRuane:3708-getrawtransaction, r=daira

add spentindex to getrawtransaction RPC results for bitcore block explorer

#3708 There are a few new `getrawtransaction` JSON result fields that the Insight block explorer depends on.

Auto merge of #4019 - bitcartel:3992_remove_obsolete_warning, r=daira

Closes #3992. Remove obsolete warning message.

Test plan: ensure that no message "%d of last 100 blocks above version %d" occurs in the log or on the metrics screen when syncing the block chain. Verify that this message *was* displayed previously for a sync from the same point.

add spentindex to getrawtransaction RPC results

Closes #3992. Remove obsolete warning message.

Co-authored-by: Simon <[email protected]>
Signed-off-by: Daira Hopwood <[email protected]>

Auto merge of #3910 - Eirik0:3909-fix-mergetoaddress, r=Eirik0

Fix z_mergetoaddress sending from ANY_SPROUT/ANY_SAPLING when the wallet contains both note types

Closes #3909

Clarify what combinations of from addresses can be used in z_mergetoaddress

Fix z_mergetoaddress sending from ANY_SPROUT/ANY_SAPLING when the wallet contains both note types

Auto merge of #4041 - Eirik0:getmigrationstatus-help, r=str4d

z_getmigrationstatus help message wording change

This is follow up from #4017

z_getmigrationstatus help message wording change

Auto merge of #4017 - bitcartel:fix_sapling_rpc_help_typo, r=Eirik0

Fix typo and clean up help message for RPC z_getmigrationstatus.

Auto merge of #4036 - Eirik0:sprout-note-entry, r=Eirik0

Replace CSproutNotePlaintextEntry with SproutNoteEntry to match Sapling

This refactoring makes the output of GetFilteredNotes safer to use, by setting the Sprout note at the point of decryption, where the corresponding address is already known, rather than having to recall it using the correct payment address.

Auto merge of #2912 - str4d:2074-main-refactor-1, r=str4d

Bitcoin Core refactoring and cleanups 1

Cherry-picked from the following upstream PRs:

- bitcoin/bitcoin#6538
- bitcoin/bitcoin#6163
- bitcoin/bitcoin#6982
- bitcoin/bitcoin#6986
- bitcoin/bitcoin#7053
- bitcoin/bitcoin#7444
- bitcoin/bitcoin#7793
  - Excluding some comments in `txmempool.h` on code we haven't yet pulled in.
- bitcoin/bitcoin#7916
- bitcoin/bitcoin#7815

Additionally, the Equihash parameters are moved into the consensus parameters, and
various other functions in `main.cpp` have `const CChainParams&` arguments added.

Part of #2074.

Auto merge of #4032 - Eirik0:remove-unused-mergetoaddress-test, r=str4d

Remove old mergetoaddress RPC test

The test wallet_mergetoaddress.py is no longer run and has been replaced by mergetoaddress_sprout.py and mergetoaddress_sapling.py. It should have been removed as part of #3619, but was not.

Globals: Explicitly pass const CChainParams& to ReceivedBlockTransactions()

Globals: Explicitly pass const CChainParams& to IsInitialBlockDownload()

Consistently use chainparams and consensusParams

Globals: Explicitly pass const CChainParams& to DisconnectBlock()

Globals: Explicit const CChainParams& arg for main:

- ContextualCheckBlockHeader
- ContextualCheckBlock

Globals: Explicitly pass const CChainParams& to ContextualCheckTransaction()

Globals: Explicit const CChainParams& arg for main:

- CheckBlockHeader
- CheckBlock

Globals: Explicitly pass const CChainParams& to IsStandardTx()

Globals: Remove Zcash-specific Params() calls from main.cpp

Break circular dependency main ↔ txdb

Break the circular dependency between main and txdb by:

- Moving `CBlockFileInfo` from `main.h` to `chain.h`. I think this makes
  sense, as the other block-file stuff is there too.

- Moving `CDiskTxPos` from `main.h` to `txdb.h`. This type seems
  specific to txdb.

- Pass a functor `insertBlockIndex` to `LoadBlockIndexGuts`. This leaves
  it up to the caller how to insert block indices.

Zcash: This does not actually break the circular dependency for us yet, as we
still need to pull in bitcoin/bitcoin#7756 and bitcoin/bitcoin#7904.

Explicitly pass CChainParams to ConnectBlock

Pass CChainParams to DisconnectTip()

Globals: Explicitly pass const CChainParams& to UpdateTip()

[doc] Fix doxygen comments for members

Improve block validity/ConnectBlock() comments

Previously didn't make clear that the ContextualCheckBlock* functions
meant the block headers as context - not the UTXO set itself - and that
ConnectBlock() also did UTXO-related validity checks (in the future we
may split that functionality into a separate UTXO-specific contextual
check block function).

Also, reordered to put validity checks first for better readability.

Globals: Remove a bunch of Params() calls from main.cpp:

1) Chainparams: Explicit CChainParams arg for main:

-AcceptBlock
-AcceptBlockHeader
-ActivateBestChain
-ConnectTip
-InitBlockIndex
-LoadExternalBlockFile
-VerifyDB parametric constructor

2) Also pickup more Params()\. in main.cpp

3) Pass nPruneAfterHeight explicitly to new FindFilesToPrune() in main.cpp

Chainparams: Explicit CChainParams arg for miner:

-BitcoinMiner
-CreateNewBlock
-GenerateBitcoins
-ProcessBlockFound

Chainparams: Explicit CChainParams arg for main (pre miner):

-ProcessNewBlock
-TestBlockValidity

Globals: Make AcceptBlockHeader static (Fix #6163)

..and at the same time prevent AcceptBlockHeader() from calling global function Params()

Globals: Explicit Consensus::Params arg for main:

-CheckBlockIndex
-DisconnectTip
-GetTransaction
-InvalidateBlock
-ProcessGetData
-ReadBlockFromDisk

Move Equihash parameters into consensus params

Add some const declarations where they are appropriate.

Declare some arguments of functions as "const" pointers where they are
not meant to be modified.

Replace CSproutNotePlaintextEntry with SproutNoteEntry to match Sapling

Auto merge of #4031 - Eirik0:2.0.6-author-aliases, r=str4d

Update author aliases

Alphabetized the existing aliases and added:

- 'Eirik0': 'Eirik Ogilvie-Wigley'
- 'mdr0id': 'Marshall Gaucher'
- 'paveljanik': 'Pavel Janik'
- 'zebambam': 'Benjamin Winston'

Auto merge of #4014 - zebambam:add_bitcoinabc_responsible_disclosure_relationship, r=Eirik0

Fixes #4013, added BitcoinABC as a disclosure partner

I've tried to avoid using "partner" as a term in general because it's not accurate, but it's quicker to type. We intend to share any vulns we become aware of in our bitcoin code with at least BitcoinABC.

Remove old mergetoaddress RPC test

Update author aliases

Auto merge of #4008 - Eirik0:3971-fix-comment, r=Eirik0

Fix tree depth in comment

Closes #3971

Fix typo and clean up help message for RPC z_getmigrationstatus.

Auto merge of #4016 - Eirik0:release-v2.0.5-2, r=Eirik0

Release v2.0.5 2

Update ZIP reference

make-release.py: Updated release notes and changelog for 2.0.5-2.

make-release.py: Updated manpages for 2.0.5-2.

make-release.py: Versioning changes for 2.0.5-2.

Auto merge of #4009 - Eirik0:2.0.5-2_release_notes, r=Eirik0

Notable changes for v2.0.5-2

Release notes wording and punctuation

Notable changes for v2.0.5-2

Fixes #4013, added BitcoinABC as a disclosure partner

Auto merge of #4002 - Eirik0:migration-status-info, r=Eirik0

Add Logging and persist async operation for Sapling migration

Currently zcashd will automatically remove the last async migration operations when it reaches the height where it sends the transactions it just made. This is not in alignment with other async operations, which are not removed until a node is restarted or a user calls `z_getoperationresult`. This PR removes the calls to pop the operations so that they can be accessed and reviewed later. In this PR I also correct the operation's `amount_migrated` field to exclude the transaction fee (this field existed for debugging purposes, but should be consistent with `z_getmigrationstatus`), and have included the list of migration txids in the operation's result (this is similar to the async rpcs such as `z_sendmany`).

Documentation: none needed.

Test plan:
* After migration transactions have been created, list the opids and call `z_getoperationresult` on them.
* Check that the operations' `amount_migrated` fields exclude the fee.

Fix LogPrint statements

Add logging for Sprout to Sapling migration transaction generation

Do not automatically remove async migration operations and return txids

Auto merge of #4005 - daira:migration-expiration, r=Eirik0

 Change expiry delta for migration transactions to 450 blocks

Documentation: to be done as part of the ZIP 308 update (zcash/zips#229).

Test plan: manually check that migration transactions have the expected expiry height, using getrawtransaction $TXID 1.

Closes #3999

Fix tree depth in comment

Fix cosmetic spacing issue in z_setmigration help.

Signed-off-by: Daira Hopwood <[email protected]>

Test the expiry height of migration transactions.

Signed-off-by: Daira Hopwood <[email protected]>

Change expiry delta for migration transactions to 450 blocks.

Signed-off-by: Daira Hopwood <[email protected]>

Repair calls to TransactionBuilder from tests.

Signed-off-by: Daira Hopwood <[email protected]>