Previously disk corruption would cause an assert instead of an exception.

f1544700 [contrib] Remove reference to sf and add doc to verify.sh (MarcoFalke)
182bec42 contrib: remove hardcoded version from verify.sh (Wladimir J. van der Laan)
c907f4d5 doc: Update release process (Wladimir J. van der Laan)

Closes #7595 as by removing the hardcoded version number from
`verify.sh`.

The actual release process quite diverged from what was written here,
also clarify things a bit.

869cf123 dbwrapper: Move `HandleError` to `dbwrapper_private` (Wladimir J. van der Laan)
b69836d6 dbwrapper: Pass parent CDBWrapper into CDBBatch and CDBIterator (Wladimir J. van der Laan)
878bf480 dbwrapper: Remove CDBWrapper::GetObfuscateKeyHex (Wladimir J. van der Laan)
74f7b127 dbwrapper: Remove throw keywords in function signatures (Wladimir J. van der Laan)

HandleError is implementation-specific.

Pass parent wrapper directly instead of obfuscation key. This
makes it possible for other databases which re-use this code
to use other properties from the database.

Add a namespace dbwrapper_private for private functions to be used
only in dbwrapper.h/cpp and dbwrapper_tests.

It is an unnecessary method as it is used only two times
and only internally, and the whole implementation is
HexStr(obfuscate_key).

Using throw() specifications in function signatures is not only
not required in C++, it is considered deprecated for
[various reasons](https://stackoverflow.com/questions/1055387/throw-keyword-in-functions-signature).
It is not implemented by any of the common C++ compilers. The usage is
also inconsistent with the rest of the source code.

f59dceb4 qt: Fix out-of-tree GUI builds (Wladimir J. van der Laan)

Without this patch:

- When I compile the GUI from the bitcoin directory itself, it works as
  expected.

- When I build the GUI in an out-of-tree build, I cannot get it to
  select tabs. When I click, say the "Receive" tab nothing happens,
  the button selects but it doesn't switch the page. The rest - even
  the debug window - seems to work.

See full discussion here:
https://github.com/bitcoin/bitcoin/pull/7911#issuecomment-212413442

This turned out to be caused by a mismatch in the arguments to moc,
preventing it from finding `bitcoin-config.h`. Fix this by passing
`$(DEFAULT_INCLUDES)` to it, which gets set to the appropriate
path by autoconf itself.

a4625acb leveldb: integrate leveldb into our buildsystem (Cory Fields)

57704499 CBase58Data::SetString: cleanse the full vector (Kaz Wesley)

3a99fb2c Fix headers announcements edge case (Suhas Daftuar)

9f7336b4 [Wallet] slightly refactor GetOldestKeyPoolTime() (Jonas Schnelli)

SetString seems to be passing the length of the wrong variable to
memory_cleanse, resulting in the last byte of the temporary buffer not being
securely erased.

807fa47a Tests: Fix deserialization of reject messages (Suhas Daftuar)

fa243293 [contrib] verify-commits: Add MarcoFalke fingerprint (MarcoFalke)

176869f9 Explicitly pass CChainParams to ConnectBlock (face)
d0a6353d Pass CChainParams to DisconnectTip() (face)
764d2377 Globals: Explicitly pass const CChainParams& to UpdateTip() (Jorge Timón)

220f950a Fix for incorrect locking in GetPubKey() (keystore.cpp) (Yuri Zhykin)

d39f5b42 net: disable resolving from storage structures (Cory Fields)
36756992 net: resolve outside of storage structures (Cory Fields)
a98cd1fc net: manually resolve dns seed sources (Cory Fields)
e9fc71e5 net: require lookup functions to specify all arguments (Cory Fields)

Previously we would assert that if every block in vBlockHashesToAnnounce is in
chainActive, then the blocks to be announced must connect.  However, there are
edge cases where this assumption could be violated (eg using invalidateblock /
reconsiderblock), so just check for this case and revert to inv-announcement
instead.

CNetAddr/CService/CSubnet can no longer resolve DNS.

Rather than allowing CNetAddr/CService/CSubNet to launch DNS queries, require
that addresses are already resolved.

This greatly simplifies async resolve logic, and makes it harder to
accidentally leak DNS queries.

Note: Some seeds aren't actually returning an IP for their name entries, so
they're being added to addrman with a source of [::].

This commit shouldn't change that behavior, for better or worse.

Assume that reject messages for blocks or transactions due to reason
REJECT_MALFORMED will not include the hash of the block or tx being rejected.

leveldb's buildsystem causes us a few problems:
- breaks out-of-tree builds
- forces flags used for some tools
- limits cross builds

Rather than continuing to add wrappers around it, simply integrate it into our
build.

fa10ce6a Move ui_interface.cpp to libbitcoin_server_a_SOURCES (MarcoFalke)
fabbf80f [ui] Move InitError, InitWarning, AmountErrMsg (MarcoFalke)

It is only needed by bitcoind and bitcoin-qt

0b25a9fb [ZMQ] append a message sequence number to every ZMQ notification (Jonas Schnelli)
de821d56 [ZMQ] refactor message string (Jonas Schnelli)