There is a circular dependency that is currently hidden in CMake builds.
chainparams.cpp from common requires primities/transactions.cpp
from bitcoinconsensus which also depends on common. This is
hidden in every other build by linking common before linking
bitcoinconsensus.

By moving primitives/transaction.cpp from bitcoinconsensus to
common, we can keep it and primitives/block.cpp together in one
library. This will remove the circular dependency mentioned above.

Discussed previously with Fabien.

ninja
ninja check
ninja bitcoin-bench

In the cmake build's src directory after successful building:

nm -D --defined-only libbitcoinconsensus.so

Confirm that the following lines appear:

0000000000009100 T bitcoinconsensus_verify_script
00000000000090b0 T bitcoinconsensus_verify_script_with_amount
0000000000008170 T bitcoinconsensus_version

Confirm gitian builds are successful