autotools: fix subdir m4 check, never let it return -I

If BDB_CPPFLAGS returns only "-I", the next argument sent to the preprocessor
is treated as a path. There are 2 fixes here:

1. Check in CPPFLAGS, as a user might have manually passed a path to check.
2. Ensure the value is not empty before setting BDB_CPPFLAGS to "-I value"
autotools: fix the Makefile.include to be safely included anywhere.

This way we can reuse rules rather than duplicating them.

Conflicts:
	src/qt/Makefile.am
	src/test/Makefile.am
win32: add version info to bitcoind.exe

TODO: Add icon info
win32: only check for windres for win32, and fail if missing
Bugfix: nHeightLast instead of nHeightFirst
fix extract_strings_qt.py

- a recent pull changed xgettext to gettext, this is reverted here
Master pull-tester-driver python script
Rename BitcoinjBitcoindComparisonTool just BitcoindComparisonTool
Tweaks to the top-level pull-tester script
Use 'low S' as malleability breaker rather than 'even S'
fix some printf -> LogPrintf leftovers
Skip unspendable outputs in consistency check
GUI: fix sendcoinsdialog/sendcoinsentry
internal miner: move 2 globals from main to miner

- moves 2 global variables from main.cpp/h to miner.cpp/h
- also removes 2 unneded includes in miner.cpp, that come from miner.h
  already
Removes obselete HomeBrew patches

Conflicts:
	contrib/homebrew/bitcoin.qt.pro.patch
fix
Inline signature serializer

Instead of building a full copy of a CTransaction being signed, and
then modifying bits and pieces until its fits the form necessary
for computing the signature hash, use a wrapper serializer that
only serializes the necessary bits on-the-fly.

This makes it easier to see which data is actually being hash,
reduces load on the heap, and also marginally improves performances
(around 3-4us/sigcheck here). The performance improvements are much
larger for large transactions, though.

The old implementation of SignatureHash is moved to a unit tests,
to test whether the old and new algorithm result in the same value
for randomly-constructed transactions.
several small Qt-related fixes

- make BitcoinGUI::showPaymentACK() use a reference for msg and use our
  own GUIUtil::HtmlEscape() function
- ensure QTimer usage in clientmodel is the same as in walletmodel
- remove an unneeded debug message in walletframe
- flag some parameters as unused in DebugMessageHandler()
- small code formatting changes
Bitcoin-Qt: add missing change from branch Qt_misc

- remove an unneeded debug message in walletframe
Bitcoin-Qt: translation fixes in sendcoins

- remove some unneeded translatable strings from sendcoinsentry.ui file and
  rename some elements for better readability
- optimize string prorcessing in SendCoinsDialog::on_sendButton_clicked()
- make all UI labels for secure payments plain text and move the settings
  to sendcoinsentry.ui file
- remove unneeded button and default button definiton from warning message
  boxes
- remove fixed font-size when sending coins to an address with label and
  use monospace font for addresses

Conflicts:
	src/qt/sendcoinsdialog.cpp