Shorten the arg log line.

Switch from `sh_out_logged` to `sh_log`.

Remove portions of `./doc/release-process.md` now implemented in `make-release.py`.

Modify the `APPROX_RELEASE_HEIGHT`.

Generalize buildnum patching for both `clientversion.h` and `configure.ac`.

Implement the IS_RELEASE rule for betas.

Implement `PathPatcher` abstraction, `clientversion.h` rewrite, and build numbering w/ unittests.

Exit instead of raising an unexpected exception, since it's already logged.

Make sure to pull the latest master.

Add an option to run against a different repo directory.

Make --help output clean by not running self-test.

Always run unittests prior to actual runs.

Make SystemExit errors less redundant in output; verify clean git status on master.

Add version sorting, assert that RELEASE_PREV is the most recent release.

Add support for beta/rc release versions.

Add all of the zcash release tags in my current repo as positive test vector.

Add another case from debian-style versions.

Update version spec by altering test; also update regex to pass single 0 digits in major/minor/patch.

Start on a make-release.py script; currently just arg parsing and unittests [unittests fail].

Auto merge of #2372 - str4d:2355-connectblock-bench, r=nathan-at-least

Benchmark for calling ConnectBlock on a block with many inputs

Requires placing `block-107134.tar.gz` (containing the block, and a fake CoinsDB containing its inputs) into the base directory of the repository.

To facilitate generation of the fake CoinsDB, an additional field `valuesZat` has been added to `getrawtransaction` containing the integer number of zatoshis instead of a decimal number of ZEC.

Closes #2355.

Auto merge of #2281 - str4d:release-version-handling, r=nathan-at-least

Release version handling

- Convert Zcash versions to Debian format for packaging
- [manpage] Handle build numbers in versions

Auto merge of #2376 - str4d:2271-miner-tests, r=nathan-at-least

Regenerate miner tests

Closes #2271.

Add a benchmark for calling ConnectBlock on a block with many inputs

Requires placing block-107134.tar.gz (containing the block, and a fake CoinsDB
containing its inputs) into the base directory of the repository. This can be
generated using qa/zcash/create_benchmark_archive.py (see the script for usage
details).

To facilitate generation of the fake CoinsDB, an additional field 'valueZat' has
been added to 'getrawtransaction' containing the integer number of zatoshis
instead of a decimal number of ZEC.

Closes #2355.

Auto merge of #2200 - str4d:1884-tx-priority-test, r=nathan-at-least

Fix prioritisetransaction RPC test

Part of #1884.

Regenerate miner tests

Closes #2271.

Auto merge of #2293 - daira:alerts-for-2017-04-11.a, r=nathan-at-least

Alerts for 2017-04-11.a security issue (fixed in 1.0.8-1)

Alert 1002 (versions 1.0.0-1.0.2 inclusive).
Alert 1003 (versions 1.0.3-1.0.8 inclusive).

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

Auto merge of #2369 - nathan-at-least:build-toolchain-logging, r=nathan-at-least

Print toolchain versions before making depends, autoconf, or main build.

Closes #2218.

Note: I spent a half hour attempting to have make do this so that people not using `build.sh` would get the same output. There's no easy way I found to ensure that output occurs first and regardless of the result of other build rules.

Auto merge of #2340 - ioptio:master, r=nathan-at-least

wallet backup instructions

closes #2178

Auto merge of #2338 - amgadabdelhafez:patch-1, r=nathan-at-least

Update timedata.cpp to fix error message string from bitcoin core to zcash

Changing error message string from bitcoin core to zcash. fixes issue #2336

Auto merge of #2297 - str4d:2274-apoptosis, r=nathan-at-least

Implement automatic shutdown of deprecated Zcash versions

Closes #2274.

Auto merge of #2251 - str4d:2210-tor-quotedstring-escapes, r=nathan-at-least

Handle escapes in Tor QuotedStrings

Also adds a missing copyright header.

Closes #2210.

Auto merge of #2362 - daira:2361.disable-proton-by-default, r=nathan-at-least

Disable building Proton by default

fixes #2361

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

Show toolchain versions in build.sh.

specify exportdir being within homedirectory

Wrap messages nicely on metrics screen

Implement automatic shutdown of deprecated Zcash versions

Closes #2274.

str4d and Ariel's suggestions

Disable building Proton by default.

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

Update timedata.cpp

Capitalizing the 'z' in "Zcash" string

typo and rewording edits

wallet backup instructions

Update timedata.cpp

Changing error message string from botcoin core to zcash

Alert 1003 (versions 1.0.3-1.0.8 inclusive).

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

Alert 1002 (versions 1.0.0-1.0.2 inclusive).

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

Check that >3-digit octals are truncated.

Correctly handle three-digit octals with leading digit 4-7

Address Daira's further comments

Auto merge of #2270 - bitcartel:2269_change_regtest_port, r=daira

Change regtest port to 18344.  Closes #2269.

Address Daira's comments

[manpage] Handle build numbers in versions

Convert Zcash versions to Debian format

torcontrol: Add missing copyright header

torcontrol: Handle escapes in Tor QuotedStrings

https://trac.torproject.org/projects/tor/ticket/14999 is tracking an encoding
bug with the Tor control protocol, where many of the QuotedString instances that
Tor outputs are in fact CStrings, but it is not documented which ones are which.

https://spec.torproject.org/control-spec section 2.1.1 provides a future-proofed
rule for handing QuotedStrings, which this commit implements.

Auto merge of #2205 - arcalinea:1804_zcash_addresses_in_help, r=daira

Change help text examples to use Zcash addresses

Closes #1804 and #2109

Ensures command line help text addresses are Zcash addresses.

Didn't change the text strings that say bitcoin or btc, leaving that for #2150 to close

Did I get all the addresses?

Change regtest port to 18344.  Closes #2269.

Auto merge of #2265 - daira:2263.fix-rpc-error-reporting, r=daira

Fix an error reporting bug due to BrokenPipeError and ConnectionResetError not existing in Python 2

refs #2263

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

Auto merge of #2189 - bitcartel:1.0.7_with_apache_qpid, r=bitcartel

Add AMQP 1.0 support via Apache Qpid Proton C++ API 0.17.0

Closes #2188

Fix proton build issue with debian jessie, as used on CI servers.

Debian 8 stable ships with gcc 4.9.2 and cmake 3.0.2.  Previously
the depends package used CMAKE_CXX_STANDARD to tell cmake to use
C++11, but the option requires cmakes 3.1+.  To resolve the issue
we now update relevant CMakeLists.txt and set CMAKE_CXX_FLAGS.

Poll on getblocktemplate result rather than use bare sleep to avoid race condition.

Fix an error reporting bug due to BrokenPipeError and ConnectionResetError not existing in Python 2.
refs #2263

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

[Direct master commit] Fix a release snafu in debian version string.

Auto merge of #2260 - nathan-at-least:release-v1.0.8-1, r=nathan-at-least

Release v1.0.8 1

Fix the previous incorrect fix to the manpage.

Undo a documentation error due to an automated tool.

Fix a release-process.md bug; commit results changelog + debian updates.

Commit the changes from gen-manpages.sh, except manually tweak the version strings.

Bump version numbers for v1.0.8-1.

Auto merge of #2259 - nathan-at-least:simplify-joinsplit-priority2, r=ebfull

simplify joinsplit priority

Add check for JoinSplit priority as calculated by CCoinsViewCache::GetPriority.

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

Simplify JoinSplit priority calculation. refs 1896

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

Add unit test for security issue 2017-04-11.a

Update tests to check actual infinity as well as INF_FEERATE

Don't rely on a finite upper bound on fee rate or priority.

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

Auto merge of #2241 - str4d:2238-benchmark-timeouts, r=ageis

Use a larger -rpcclienttimeout for slow performance measurements

Closes #2238.

Use a larger -rpcclienttimeout for slow performance measurements

Closes #2238.

Auto merge of #2213 - daira:2211.release-1.0.8, r=daira

2211.release-1.0.8

Release 1.0.8. refs #2211

Update authors, release notes, and Debian package metadata.

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

Generate man pages.

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

Improvement to release process doc.

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

Update version numbers.

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

Auto merge of #2177 - str4d:2061-tor-ephemeral-hs, r=daira

Tor ephemeral hidden services

Cherry-picked from the following upstream PRs:

- bitcoin/bitcoin#6503 (included to reduce merge conflicts)
- bitcoin/bitcoin#6639
- bitcoin/bitcoin#6643
- bitcoin/bitcoin#7090
- bitcoin/bitcoin#7035
- bitcoin/bitcoin#7170
- bitcoin/bitcoin#7218 (non-QT part)
- bitcoin/bitcoin#7313
- bitcoin/bitcoin#7438
- bitcoin/bitcoin#7553
- bitcoin/bitcoin#7637
- bitcoin/bitcoin#7683
- bitcoin/bitcoin#7813
- bitcoin/bitcoin#7703
- bitcoin/bitcoin#8203
- bitcoin/bitcoin#9004
- bitcoin/bitcoin#9234
- bitcoin/bitcoin#9911 (partial)

Closes #2061.

Auto merge of #2203 - arcalinea:2142_security_log_message, r=daira

Add message directing users to security guide

Addresses #2142, which was blocking on updates to zcash/support/security.html. That page has now been added, so this message directing users to the site can be included.

It displays in the zcash-cli --help and --version message text, and on the zcashd metrics screen.

torcontrol: Log invalid parameters in Tor reply strings where meaningful

Line-wrap privacy notice. Use <> around URL and end sentence with '.'.
Include privacy notice in help text for zcashd -help.

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

torcontrol: Check for reading errors in ReadBinaryFile

This ensures that ReadBinaryFile never returns exactly TOR_COOKIE_SIZE bytes if
the file was larger than that.

Add --disable-proton flag to build.sh.  Proton has build/linker issues with gcc 4.9.2 and requires gcc 5.x.

Add AMQP 1.0 support via Apache Qpid Proton C++ API 0.17.0

torcontrol: Fix ParseTorReplyMapping

- Ignore remaining input if it is an OptArguments
- Correctly handle escapes

torcontrol: Add unit tests for Tor reply parsers

torcontrol: Improve comments

Make some global variables less-global (static)

Prevent -Wshadow warnings with gcc versions 4.8.5, 5.3.1 and 6.2.1.

Zcash: partial (ignoring src/test/script_tests.cpp due to merge conflicts)

torcontrol: Explicitly request RSA1024 private key

When generating a new service key, explicitly request a RSA1024 one.

The bitcoin P2P protocol has no support for the longer hidden service names
that will come with ed25519 keys, until it does, we depend on the old
hidden service type so make this explicit.

See #9214.

Clarify `listenonion`

> This new feature is enabled by default if Bitcoin Core is listening, and a connection to Tor can be made. It can be configured with the -listenonion, -torcontrol and -torpassword settings. To show verbose debugging information, pass -debug=tor.

But it is correct to say that the feature is enabled *regardless* of whether a connection to Tor can be made.

I propose to clarify that so that users can eliminate these in their logs (when `listen=1` and no Tor).
And I think it's okay to clarify about the `listen` option, because on several occasions when I read this before I always assumed `listening` meant `server=1` which cost me a lot of time in troubleshooting.

```
2016-10-24 06:19:22.551029 tor: Error connecting to Tor control socket
2016-10-24 06:19:22.551700 tor: Not connected to Tor control port 127.0.0.1:9051, trying to reconnect
```

0.12.1

Clarify documentation for running a tor node

Previous wording suggested that no additional setup was required for a
tor hidden service to be created.

tor: Change auth order to only use HASHEDPASSWORD if -torpassword

Change authentication order to make it more clear (see #7700).

- If the `-torpassword` option is provided, force use of
  `HASHEDPASSWORD` auth.

- Give error message if `-torpassword` provided, but
  `HASHEDPASSWORD` auth is not available.

- Give error message if only `HASHEDPASSWORD` available, but
  `-torpassword` not given.

[doc] Update port in tor.md

Tor Browser Bundle spawns the Tor process and listens on port 9150, it doesn't randomly pick a port.

[ci skip]
(cherry picked from commit 1b63cf98347b2a62915425576930f55c2126c2ff)

Fix torcontrol.cpp unused private field warning

Fix memleak in TorController [rework]

It looks like, TorController::disconnected_cb(TorControlConnection&
conn) gets called multiple times which results in multiple event_new().

Avoid this by creating the event only once in the constructore, and
deleting it only once in the destructor (thanks to Cory Fields for the
idea).

Replaces the fix by Jonas Schnelli in #7610, see discussion there.

Remove vfReachable and modify IsReachable to only use vfLimited.

We do not know that a class of Network is reachable, only that it is not.