Auto merge of #1980 - bitcartel:fix_z_sendmany_logging_bug_txid, r=ebfull

Fix bug where z_sendmany logged incorrect txid fragment when sending from taddr

Closes #1987.

Auto merge of #1977 - bitcartel:1967_z_sendmany_debug_age_of_notes, r=ebfull

Fixes #1967 by adding age of note to z_sendmany logging.

Auto merge of #1976 - bitcartel:1964_z_sendmany_exception_handling, r=ebfull

Fixes #1964 to catch general exception in z_sendmany and catch exceptions as reference-to-const.

Auto merge of #1974 - str4d:1948-debian-lint, r=ebfull

Fix Debian lintian warnings

Closes #1948.

Auto merge of #1973 - str4d:1943-remove-extraneous-bins-from-dist, r=ebfull

Exclude test binaries from make install

Closes #1943.

Auto merge of #1972 - str4d:1950-ensure-messages-are-rendered, r=ebfull

Ensure messages are rendered in the metrics UI

Part of #1950.

Auto merge of #1942 - str4d:1940-threaded-createjoinsplit, r=ebfull

Extend createjoinsplit to benchmark parallel JoinSplits

Closes #1940

Auto merge of #1935 - arcalinea:1934_user_followup_in_release_process, r=ebfull

Update release process to check in with users who opened resolved issues

Updates release-process.md to include a step to check in with users who opened `user support` issues resolved in the release.
Resolves #1934.

Auto merge of #1991 - bitcartel:1960_z_getoperation_include_call_details, r=ebfull

Fixes #1960: z_getoperationstatus/result now includes operation details.

Auto merge of #2016 - ioptio:issue-template, r=str4d

Create ISSUE_TEMPLATE.md

Auto merge of #2023 - str4d:create-config-file-in-perf-tests, r=str4d

Create empty zcash.conf during performance measurements

Fixes benchmarking regression caused by #2013.

Create empty zcash.conf during performance measurements

Auto merge of #2015 - bitcartel:1780_sort_getoperationstatus, r=ebfull

Closes #1780. Result of z_getoperationstatus now sorted by operation's creation time.

Generate Debian control file to fix shlibs lint

Debian package lint

- Tweak description synopsis to make Debian happy
- Put bash completion files in correct directory
- Add a manpage for zcash-fetch-params

Fixes #1960: z_getoperationstatus/result now includes operation details.

Auto merge of #2013 - str4d:2002-require-zcash-conf, r=ebfull

Require zcash.conf to start zcashd

Closes #2002.

Auto merge of #2021 - str4d:1539-fix-test-ordering-again, r=ebfull

Fix gtest ordering broken by #1949

Part of #1539

Fix gtest ordering broken by #1949

Part of #1539

Auto merge of #2019 - bitcartel:remove-bom, r=ebfull

Remove UTF-8 BOM 0xefbbbf from zcash.conf to avoid problems with command line tools

Closes #2018

Remove UTF-8 BOM efbbbf from zcash.conf to avoid problems with command line tools

move template to subdirectory, fix typo, include prompt under describing issue section, include uploading file directly to github ticket as option for sharing logs

Create ISSUE_TEMPLATE.md

Closes #1780. Result of z_getoperationstatus now sorted by creation time of operation

Show a friendly message explaining why zcashd needs a zcash.conf

Throw an error if zcash.conf is missing

An empty zcash.conf is sufficient to bypass this error.

Auto merge of #2009 - bitcartel:2008_update_copyright_to_2017, r=str4d

Bump COPYRIGHT_YEAR from 2016 to 2017.

Closes #2008

Bump COPYRIGHT_YEAR from 2016 to 2017.

Auto merge of #1949 - str4d:1810-fix-disablewallet, r=str4d

Fix --disablewallet

Closes #1810.

Auto merge of #1986 - tromer:createjoinsplit-startprofiling, r=ebfull

CreateJoinSplit: add start_profiling() call

This solves the problem of profiling output from the CreteJoinSplit benchmarking binary displaying nonsensical large time values.

CreateJoinSplit: add start_profiling() call

This solves the problem of profiling output displaying nonsensical large time values.

Auto merge of #1981 - ebfull:fixing-homu-merge, r=ebfull

Add 'CreateJoinSplit' standalone utility to gitignore.

Closes #1926 (I'm going to use this little change to test the homu code change.)

Handle newlines in UI messages

Add 'CreateJoinSplit' standalone utility to gitignore.

Strip out the SECURE flag in metrics UI so message style is detected

Trigger metrics UI refresh on new messages

Fixes a bug where the unsigned transaction was logged by z_sendmany
after a successful sign and send, meaning that the logged hash fragment
would be different from the txid logged by "AddToWallet".  This issue
occured when sending from transparent addresses, as utxo inputs must be
signed.  It did not occur when sending from shielded addresses.

Fixes #1967 by adding age of note to z_sendmany logging.

Fixes #1964 to catch general exception in z_sendmany and catch
exceptions as reference-to-const.

Exclude test binaries from make install

Closes #1943.

Auto merge of #1946 - str4d:blockchain-stats-num-commitments, r=ebfull

Add total number of commitments to getblockchaininfo

Add test for IncrementalMerkleTree::size().

Add a tool for profiling the creation of JoinSplits

Only run wallet tests when wallet is enabled

Only enable getblocktemplate when wallet is enabled

Release notes, authors, changelog

Update man pages

Update version to 1.0.4

Add total number of commitments to getblockchaininfo

Auto merge of #1939 - str4d:1933-fix-bug, r=str4d

Only increment new notes on reindex

Addresses another issue in #1904.

When an existing one of our notes was found again, its cache was reset and it was re-witnessed. This would cause encountered notes to get out-of-sync with the otherwise-ignored newer notes, which could be a problem if the wallet data happens to be written out during a reindex.

Update payment API docs to recommend -rescan for fixing witness errors

Extend createjoinsplit to benchmark parallel JoinSplits

Closes #1940

Auto merge of #1938 - ebfull:g2-subgroup-check, r=str4d

Additional well-formedness check for G2 elements

libsnark currently checks that G₁ and G₂ elements are well-formed by ensuring that they satisfy their respective curve equations, and although this is enough for G₁ (which is instantiated as an order r curve E/F_p: y^2 = x^3 + b), G₂ is the order r *subgroup* of the composite order r(2q-r) curve E'/Fp²: y^2 = x^3 + b/e constructed via a sextic twisting isomorphism. This means we need to ensure these points are order r as well.

None of the proofs on the Zcash blockchain violate this check, and it may not even be possible for them to violate this check (bilinearity is not preserved). Let's be cautious and do it anyway.

Auto merge of #1892 - str4d:1749-disable-js-verification-before-checkpoints, r=str4d

Skip JoinSplit verification before the last checkpoint

Part of #1749

Fix bug in IncrementNoteWitness()

Rewrite reindex test to check beyond the max witness cache size

Extract block-generation wallet test code into a function

Fix bug in wallet tests

Check that E' points are actually in G2 by ensuring they are of order r.

Update release process to check in with users who opened resolved issues

Only check cache validity for witnesses being incremented or decremented

Fixes the bug resulting from #1904.

Auto merge of #1904 - str4d:1749-write-witness-cache-with-best-block, r=ebfull

Write witness caches when writing the best block

For steady-state operation, this reduces the average time between wallet disk
writes from once per block to once per hour.

On -rescan, witness caches are only written out at the end along with the best
block, increasing speed while ensuring that on-disk state is kept consistent.

Witness caches are now never recreated during a -reindex, on the assumption that
the blocks themselves are not changing (the chain is just being reconstructed),
and so the witnesses will remain valid.

Part of #1749.

Make the test pass by fixing the bug!

Add a reindex test that fails because of a bug in decrementing witness caches

Ref: https://github.com/zcash/zcash/pull/1904#issuecomment-265992988

Auto merge of #1930 - arcalinea:revise-release-note-generation, r=str4d

Gather release notes from previous release to HEAD

Release-notes.py used to depend on release already being tagged--now gathers release notes from previous tagged release up to current HEAD.

Gather release notes from previous release to HEAD

Also update release-process.md to replace git shortlog command with
release-notes.py script.

Auto merge of #1918 - bitcartel:timeout_cpu_throttling, r=str4d

Increase timeout as laptops on battery power have cpu throttling.

Merge this last as other PRs also update the test file `wallet_protectcoinbase.py`.

Auto merge of #1895 - bitcartel:1857_tx_priority, r=str4d

Closes #1857. Fixes bug where a transaction sending from a zaddr would have a priority of zero.

Transactions sent from a zaddr should now be mined sooner as they no longer have a priority of zero

Skip JoinSplit verification before the last checkpoint

Part of #1749

Auto merge of #1870 - str4d:1749-benchmark-rescanning, r=str4d

Add benchmarks for rescan components

Part of #1749.

Auto merge of #1919 - ebfull:abstract-verification, r=str4d

Isolate verification to a ProofVerifier context object that allows verification behavior to be tuned by the caller.

This is an alternative foundation for #1892, i.e., #1892 will have to be changed if this PR is accepted.

I think this is a safer approach because it allows us to isolate verification behavior to a single object. This will come in handy when @arielgabizon finishes the batching code.

ASSERT_TRUE -> ASSERT_FALSE

Add JS to second block to ensure witnesses are incremented

Generate JS for trydecryptnotes, make number of addresses a variable

Fix indentation

Auto merge of #1642 - str4d:add-solver-rate-to-getmininginfo, r=str4d

Add getlocalsolps and getnetworksolps RPC calls, show them in getmininginfo

Auto merge of #1902 - bitcartel:1901_increase_default_settings_block_priority, r=str4d

Closes #1901. Increase default settings for max block size and space for priority transactions.

ref #1901

Auto merge of #1928 - ebfull:fix-anchor-cache-bug, r=str4d

Fix anchor cache bug

Fixes #1912.

If an anchor is removed from the cache, but didn't exist in it beforehand, it will insert a blank tree. If it's reinserted in a child cache, when the child cache flushes it will mark the treestate as entered but won't bring the valid tree with it.

Thankfully, we assert when connecting blocks so that this inconsistency won't cause us to build on a blank tree after a reorg.

Add more tests for ProofVerifier.

Rename Dummy to Disabled.

Auto merge of #1911 - bitcartel:1823_witness_does_not_have_same_anchor_as_change_input, r=str4d

With chained joinsplits, witness anchors for input notes no longer cross block boundaries

Closes #1823

Auto merge of #1909 - str4d:bash-completion, r=str4d

Bash completion

This PR pulls in bitcoin/bitcoin#8289, updates the bash completion files for use with Zcash, and bundles them into the Debian package.

Auto merge of #1906 - str4d:1835-solaris-miniupnpc-patches, r=str4d

Apply MiniUPnP patches to enable compilation on Solaris 11

These can be removed after the next MiniUPnP release.

Closes #1835.

Auto merge of #1878 - str4d:1875-non-tty-metrics-usability, r=str4d

Improve non-TTY metrics usability

Closes #1875.

Auto merge of #1905 - str4d:1831-build-script-overrides, r=str4d

Add porter dev overrides for CC, CXX, MAKE, BUILD, HOST

Replaces #1839. Closes #1831.

WitnessAnchorData only needs to store one witness per JSOutPoint.

Auto merge of #1907 - bitcartel:1903_z_sendmany_fee_parameter, r=ebfull

Closes #1903. Add fee parameter to z_sendmany.

Document behaviour of CWallet::SetBestChain

Ensure ProofVerifier cannot be accidentally copied.

Ensure cache contains valid entry when anchor is popped.

Regression test.

Isolate verification to a `ProofVerifier` context object that allows verification behavior to be tuned by the caller.

Increase timeout as laptops on battery power have cpu throttling.

Edit for grammar: "block chain"

At this point, I believe it is universally accepted that "blockchain" is one word, and should not be separated into two.

Fixes #1823. Witness anchors for input notes no longer cross block boundaries.

Add Zcash RPC commands to CLI argument completion

Always bash-complete the default account

Add bash completion files to Debian package

Change function names to not clash with Bitcoin, apply to correct binaries

bash-completion: Adapt for 0.12 and 0.13

 * separate completion for bitcoind and bitcoin-cli
 * remove RPC support from bitcoind completion
 * add completion for bitcoin-tx and bitcoin-qt
 * rely on autoloading of completions