181 Commits (bd57dcbaf2b5e5f50833912c894a1f1239ceb25b)

Author	SHA1	Message	Date
MarcoFalke	bca00942ed	Merge bitcoin/bitcoin#21731: Update msvc build to use Qt5.12.10 binaries. ... f02ca7a354 Update msvc build to use Qt5.12.10 binaries. (Aaron Clauson) Pull request description:  ACKs for top commit: hebasto: ACK f02ca7a354, I made a customized AppVeyor build with an artifact: https://ci.appveyor.com/project/hebasto/bitcoin/builds/38786868/artifacts Tree-SHA512: 0f8998a5b72bca9a08fc5ec5c40b0b79e9247486f58f74824ebf045175d9e3ce7485c16d8de574b7316e79a8433af0646870abf4b0f3b47fc35a92f63a7b5dc9	4 years ago
dplusplus1024	de17d245b7	Re-add command to install vcpkg ... `vcpkg integrate install` must be executed so that msbuild will automatically install external dependencies. It was removed in 712f95d332 It was originally added in 7644567758	4 years ago
Aaron Clauson	f02ca7a354	Update msvc build to use Qt5.12.10 binaries.	4 years ago
Pieter Wuille	cabb566123	Disable certain false positive warnings for libsecp256k1 msvc build	4 years ago
fanquake	3ececa76b7	Merge #21543: build: Enable safe warnings for msvc builds ... 7c543bc27b build: Enable safe warnings for msvc builds (Hennadii Stepanov) Pull request description: Disabling of some specific warnings no longer needed. ACKs for top commit: MarcoFalke: ACK 7c543bc27b if green practicalswift: ACK 7c543bc27b if green: compiler warnings are good, compiler safety warnings are great sipsorcery: tACK 7c543bc27b. Tree-SHA512: a8253a5a0ccc22b8622ae66efae77a163c54bbd6dbd7e28b043ad4eb474b2c8e5c7b22d3870b15a2b9b91bade0a5ae5bfc5ee765d132dc32ddbbe690fb0146d9	4 years ago
Hennadii Stepanov	7c543bc27b	build: Enable safe warnings for msvc builds	4 years ago
fanquake	9ac86bcc0d	test: remove qt byteswap compattests ... These were added as part of #9366 to fix issues with Protobuf. Now that we no-longer use Protobuf, there's no reason to maintain a duplicate set of byteswap tests for qt.	4 years ago
Hennadii Stepanov	0eabb2abed	build: Remove unused header from the build system	4 years ago
Hennadii Stepanov	df2d165ba9	qt: Add peertablesortproxy module	4 years ago
Wladimir J. van der Laan	e017a913d0	bitcoind: Add -daemonwait option to wait for initialization ... This adds a `-daemonwait` flag that does the same as `-daemon` except it, from a user perspective, backgrounds the process only after initialization is complete. This can be useful when the process launching bitcoind wants to guarantee that either the RPC server is running, or that initialization failed, before continuing. The exit code indicates the initialization result. This replaces the use of the libc function `daemon()` by a custom implementation which is inspired by the glibc implementation, but also creates a pipe from the child to the parent process for communication. An additional advantage of having our own `daemon()` implementation is that no MACOS-specific pragmas are needed anymore to silence a deprecation warning.	4 years ago
Ethan Heilman	9bd3f35003	build: adds switch for disabling random base addresses in MSVC	4 years ago
Wladimir J. van der Laan	a9335e4f12	Merge #16546: External signer support - Wallet Box edition ... f75e0c1edd doc: add external-signer.md (Sjors Provoost) d4b0107d68 rpc: send: support external signer (Sjors Provoost) 245b4457cf rpc: signerdisplayaddress (Sjors Provoost) 7ebc7c0215 wallet: ExternalSigner: add GetDescriptors method (Sjors Provoost) fc5da520f5 wallet: add GetExternalSigner() (Sjors Provoost) 259f52cc33 test: external_signer wallet flag is immutable (Sjors Provoost) 2655197e1c rpc: add external_signer option to createwallet (Sjors Provoost) 2700f09c41 rpc: signer: add enumeratesigners to list external signers (Sjors Provoost) 07b7c940a7 rpc: add external signer RPC files (Sjors Provoost) 8ce7767071 wallet: add ExternalSignerScriptPubKeyMan (Sjors Provoost) 157ea7c614 wallet: add external_signer flag (Sjors Provoost) f3e6ce78fb test: add external signer test (Sjors Provoost) 8cf543f96d wallet: add -signer argument for external signer command (Sjors Provoost) f7eb7ecc67 test: framework: add skip_if_no_external_signer (Sjors Provoost) 87a97941f6 configure: add --enable-external-signer (Sjors Provoost) Pull request description: Big picture overview in [this gist](https://gist.github.com/Sjors/29d06728c685e6182828c1ce9b74483d). This PR lets `bitcoind` call an arbitrary command `-signer=<cmd>`, e.g. a hardware wallet driver, where it can fetch public keys, ask to display an address, and sign a transaction (using PSBT under the hood). It's design to work with https://github.com/bitcoin-core/HWI, which supports multiple hardware wallets. Any command with the same arguments and return values will work. It simplifies the manual procedure described [here](https://github.com/bitcoin-core/HWI/blob/master/docs/bitcoin-core-usage.md). Usage is documented in [doc/external-signer.md]( https://github.com/Sjors/bitcoin/blob/2019/08/hww-box2/doc/external-signer.md), which also describes what protocol a different signer binary should conform to. Use `--enable-external-signer` to opt in, requires Boost::Process: ``` Options used to compile and link: with wallet = yes with gui / qt = no external signer = yes ``` It adds the following RPC methods: * `enumeratesigners`: asks <cmd> for a list of signers (e.g. devices) and their master key fingerprint * `signerdisplayaddress <address>`: asks <cmd> to display an address It enhances the following RPC methods: * `createwallet`: takes an additional `external_signer` argument and fetches keys from device * `send`: automatically sends transaction to device and waits Usage TL&DR: * clone HWI repo somewhere and launch `bitcoind -signer=../HWI/hwi.py` * check if you can see your hardware device: `bitcoin-cli enumeratesigners` * create wallet and auto import keys `bitcoin-cli createwallet "hww" true true "" true true true` * display address on device: `bitcoin-cli signerdisplayaddress ...` * to spend, use `send` RPC and approve transaction on device Prerequisites: - [x] #21127 load wallet flags before everything else - [x] #21182 remove mostly pointless BOOST_PROCESS macro Potentially useful followups: - GUI support: bitcoin-core/gui#4 - bumpfee support - (automatically) verify (a subset of) keys on the device after import, through message signing ACKs for top commit: laanwj: re-ACK f75e0c1edd Tree-SHA512: 7db8afd54762295c1424c3f01d8c587ec256a72f34bd5256e04b21832dabd5dc212be8ab975ae3b67de75259fd569a561491945750492f417111dc7b6641e77f	4 years ago
fanquake	5e531e6beb	assumptions: check C++17 assumption with MSVC ... From my reading of https://docs.microsoft.com/en-us/cpp/build/reference/zc-cplusplus?view=msvc-160 and https://devblogs.microsoft.com/cppblog/msvc-now-correctly-reports-__cplusplus/ if we set the `/Zc:__cplusplus` switch in additional options, MSVC will report the correct value for `__cplusplus`.	4 years ago
Sjors Provoost	87a97941f6	configure: add --enable-external-signer ... This option replaces --with-boost-process This prepares external signer support to be disabled by default. It adds a configure option to enable this feature and to check if Boost::Process is present. This also exposes ENABLE_EXTERNAL_SIGNER to the test suite via test/config.ini	4 years ago
fanquake	060a2a64d4	ci: remove boost thread installation ... Adjust fuzzbuzz.yml to only install the Boost components we need.	4 years ago
fanquake	06e1d7d81d	build: don't build or use Boost Thread	4 years ago
Hennadii Stepanov	c5354e4641	Fix MSVC build after gui#176	4 years ago
MarcoFalke	fa0074e2d8	scripted-diff: Bump copyright headers ... -BEGIN VERIFY SCRIPT- ./contrib/devtools/copyright_header.py update ./ -END VERIFY SCRIPT-	4 years ago
Aaron Clauson	8b99e609e7	Adjusted msvc compiler and linker settings to remove optimisations that are causing sporadic ABI issues on Visual Studio updates. ... Tidied up debug and release configuration blocks in common project file to avoid duplication. Updated appveyor config to use latest Visual Studio 2019 image. Changed appveyor config file hash to use a new version of Qt pre-compiled binaries built for Visual Studio 2019 v16.8.1. Bumped vcpkg version to tag '2020.11-1' for binary caching feature. See #20392 for related discussion.	4 years ago
Wladimir J. van der Laan	86bf3ae3b5	Merge #20202: wallet: Make BDB support optional ... d52f502b1e Fix mock SQLiteDatabases (Andrew Chow) 99309ab3e9 Allow disabling BDB in configure with --without-bdb (Andrew Chow) ee47f11f73 GUI: Force descriptor wallets when BDB is not compiled (Andrew Chow) 71e40b33bd RPC: Require descriptors=True for createwallet when BDB is not compiled (Andrew Chow) 6ebc41bf9c Enforce salvage is only for BDB wallets (Andrew Chow) a58b719cf7 Do not compile BDB things when USE_BDB is defined (Andrew Chow) b33af48210 Include wallet/bdb.h where it is actually being used (Andrew Chow) Pull request description: Adds a `--without-bdb` option to `configure` which disables the compilation of the BDB stuff. Legacy wallets will not be created when BDB is not compiled. A legacy-sqlite wallet can be loaded, but we will not create them. Based on #20156 to resolve the situation where both `--without-sqlite` and `--without-bdb` are provided. In that case, the wallet is disabled and `--disable-wallet` is effectively set. ACKs for top commit: laanwj: Code review ACK d52f502b1e Tree-SHA512: 5a92ba7a542acc2e27003e9d4e5940e0d02d5c1f110db06cdcab831372bfd83e8d89c269caff31dd5bff062c1cf5f04683becff12bd23a33be731676f346553d	4 years ago
Andrew Chow	8f7b930475	Drop the leading 0 from the version number ... Removes the leading 0 from the version number. The minor version, which we had been using as the major version, is now the major version. The revision, which we had been using as the minor version, is now the minor version. The revision number is dropped. The build number is promoted to being part of the version number. This also avoids issues where it was accidentally not included in the version number. The CLIENT_VERSION remains the same format as previous as previously, the Major version was 0 so that was never a factor in CLIENT_VERSION.	4 years ago
Andrew Chow	99309ab3e9	Allow disabling BDB in configure with --without-bdb	4 years ago
Andrew Chow	a58b719cf7	Do not compile BDB things when USE_BDB is defined	4 years ago
Wladimir J. van der Laan	132e1d897f	build: Bump master version to 0.21.99 ... Tree-SHA512: 94c258b234b2412d92f312a1b38adf17249664a9e3e321de0ff683b59a48cee192cd42da5220df0726a782d98776610f4420534b3a1c51f4cf4a0180d5835622	4 years ago
Wladimir J. van der Laan	3caee16946	Merge #19953: Implement BIP 340-342 validation (Schnorr/taproot/tapscript) ... 0e2a5e448f tests: dumping and minimizing of script assets data (Pieter Wuille) 4567ba034c tests: add generic qa-asset-based script verification unit test (Pieter Wuille) f06e6d0345 tests: functional tests for Schnorr/Taproot/Tapscript (Pieter Wuille) 3c226639eb tests: add BIP340 Schnorr signature support to test framework (Pieter Wuille) 206fb180ec --- [TAPROOT] Tests --- (Pieter Wuille) d7ff237f29 Activate Taproot/Tapscript on regtest (BIP 341, BIP 342) (Pieter Wuille) e9a021d7e6 Make Taproot spends standard + policy limits (Pieter Wuille) 865d2c37e2 --- [TAPROOT] Regtest activation and policy --- (Pieter Wuille) 72422ce396 Implement Tapscript script validation rules (BIP 342) (Johnson Lau) 330de894a9 Use ScriptExecutionData to pass through annex hash (Pieter Wuille) 8bbed4b7ac Implement Taproot validation (BIP 341) (Pieter Wuille) 0664f5fe1f Support for Schnorr signatures and integration in SignatureCheckers (BIP 340) (Pieter Wuille) 5de246ca81 Implement Taproot signature hashing (BIP 341) (Johnson Lau) 9eb590894f Add TaggedHash function (BIP 340) (Pieter Wuille) 450d2b2371 --- [TAPROOT] BIP340/341/342 consensus rules --- (Pieter Wuille) 5d62e3a68b refactor: keep spent outputs in PrecomputedTransactionData (Pieter Wuille) 8bd2b4e784 refactor: rename scriptPubKey in VerifyWitnessProgram to exec_script (Pieter Wuille) 107b57df9f scripted-diff: put ECDSA in name of signature functions (Pieter Wuille) f8c099e220 --- [TAPROOT] Refactors --- (Pieter Wuille) Pull request description: This is an implementation of the Schnorr/taproot consensus rules proposed by BIPs [340](https://github.com/bitcoin/bips/blob/master/bip-0340.mediawiki), [341](https://github.com/bitcoin/bips/blob/master/bip-0340.mediawiki), and [342](https://github.com/bitcoin/bips/blob/master/bip-0340.mediawiki). See the list of commits [below](https://github.com/bitcoin/bitcoin/pull/19953#issuecomment-691815830). No signing or wallet support of any kind is included, as testing is done entirely through the Python test framework. This is a successor to https://github.com/bitcoin/bitcoin/pull/17977 (see discussion following [this comment](https://github.com/bitcoin/bitcoin/pull/17977#issuecomment-682285983)), and will have further changes squashed/rebased. The history of this PR can be found in #19997. ACKs for top commit: instagibbs: reACK 0e2a5e448f benthecarman: reACK 0e2a5e4 kallewoof: reACK 0e2a5e448f jonasnick: ACK 0e2a5e448f almost only looked at bip340/libsecp related code jonatack: ACK 0e2a5e448f modulo the last four commits (tests) that I plan to finish reviewing tomorrow fjahr: reACK 0e2a5e448f achow101: ACK 0e2a5e448f Tree-SHA512: 1b00314450a2938a22bccbb4e177230cf08bd365d72055f9d526891f334b364c997e260c10bc19ca78440b6767712c9feea7faad9a1045dd51a5b96f7ca8146e	4 years ago
Andrew Chow	e87df82580	Add sqlite to travis and depends	4 years ago
Pieter Wuille	0664f5fe1f	Support for Schnorr signatures and integration in SignatureCheckers (BIP 340) ... This enables the schnorrsig module in libsecp256k1, adds the relevant types and functions to src/pubkey, as well as in higher-level `SignatureChecker` classes. The (verification side of the) BIP340 test vectors is also added.	4 years ago
Aaron Clauson	712f95d332	Update msvc build to use new vcpkg manifest ... The vcpkg tool has introduced a proper way to use manifests, https://devblogs.microsoft.com/cppblog/vcpkg-accelerate-your-team-development-environment-with-binary-caching-and-manifests/. This PR replaces the custom text file mechanism with the new manifest approach. It is planned that vckpg manifests will include the ability to version dependencies in the future. Dependency versions would solve a number of issues that currently require workarounds with the appveyor CI. Set vcpkg manifest version to 1 to avoid any perception it's related to any release or other version numbering.	4 years ago
Luke Dashjr	1ccb9f30c0	Move Win32 defines to configure.ac to ensure they are globally defined ... common.vcxproj used for MSVC builds	4 years ago
Sjors Provoost	3c84d85f7d	[build] msvc: add boost::process ... * AppVeyor boost-process vcpkg package. * Tell Boost linter to ignore it * Add HAVE_BOOST_PROCESS for MSVC build (bitcoin_config.h)	4 years ago
Aaron Clauson	2894e94d17	Updates msvc build to use ISO standard C++17.	4 years ago
fanquake	9952242c03	build: improve builtin_clz* detection ... The way we currently test with AC_CHECK_DECLS do not work with Clang: ```bash configure:21492: clang++-10 -std=c++11 -c -g -O2 -DHAVE_BUILD_INFO -D__STDC_FORMAT_MACROS conftest.cpp >&5 conftest.cpp💯10: error: builtin functions must be directly called (void) __builtin_clz; ^ 1 error generated. ``` This also removes the __builtin_clz() check, as we don't actually use it anywhere, and it's trvial to re-add detection if we do start using it at some point.	4 years ago
Glenn Willen	a6cb0b0c29	[gui] PSBT Operations Dialog (sign & broadcast) ... Add a "PSBT Operations" dialog, reached from the "Load PSBT..." menu item, giving options to sign or broadcast the loaded PSBT as appropriate, as well as copying the result to the clipboard or saving it to a file.	4 years ago
Pieter Wuille	ddc2419c09	Update MSVC build config for libsecp256k1	4 years ago
fanquake	8bf1540cc2	build: remove fdelt_chk backwards compatibility code ... Now that we require glibc 2.17 or later, we no longer need to check for different return types in fdelt_chk. It was changed from unsigned long int to long int in glibc 2.16 . See this commit: https://sourceware.org/git/?p=glibc.git;a=commit;h=ceb9e56b3d1f8c1922e0526c2e841373843460e2 and related issue: https://sourceware.org/bugzilla/show_bug.cgi?id=14210.	5 years ago
MarcoFalke	fa488f131f	scripted-diff: Bump copyright headers ... -BEGIN VERIFY SCRIPT- ./contrib/devtools/copyright_header.py update ./ -END VERIFY SCRIPT-	5 years ago
MarcoFalke	4eb1eeb02c	Merge #18504: build: Drop bitcoin-tx and bitcoin-wallet dependencies on libevent ... 01a3392b1b Drop bitcoin-wallet dependency on libevent (Russell Yanofsky) 0660119ac3 Drop unintended bitcoin-tx dependency on libevent (Russell Yanofsky) Pull request description: This fixes compile errors trying to build bitcoin-tx and bitcoin-wallet without libevent, which were reported by Luke Dashjr in https://github.com/bitcoin/bitcoin/issues/18465 The fix avoiding `bitcoin-tx` dependency on libevent just adds a conditional build rule. This is implemented in the first commit (more details in commit description). The fix avoiding `bitcoin-wallet` dependency on libevent requires minor code changes, because `bitcoin-wallet` (unlike `bitcoin-tx`) links against code that calls `urlDecode` / `evhttp_uridecode`. This fix is implemented in the second commit (again details in the commit description). ACKs for top commit: jonasschnelli: utACK 01a3392b1b. Tree-SHA512: d2245e912ab494cccceeb427a1eca8e55b01a0006ff93eebcfb5461ae7cecd1083ac2de443d9db036b18bdc6f0fb615546caaa20c585046f66d234937f74870a	5 years ago
Wladimir J. van der Laan	d84c9aa25d	build: Bump version to 0.20.99 ... Now that 0.20 branch has been split off, master is 0.20.99 (pre-0.21). Also clean out release notes. Tree-SHA512: bba6133ae9708f75206c8934901b9f9909a233330f4dfefb3c24175bf8e11631cdc89a5d24a22421a73083f7eb743e977db8020b87dfbd3c1e6043929a19a285	5 years ago
fanquake	9e071b0089	test: remove rapidcheck integration and tests	5 years ago
Russell Yanofsky	0660119ac3	Drop unintended bitcoin-tx dependency on libevent ... Don't include util/url.cpp to libbitcoin_util.a when libevent isn't available. This fixes a compile error trying to build bitcoin-tx without libevent reported by Luke Dashjr in https://github.com/bitcoin/bitcoin/issues/18465 Fixes #18465	5 years ago
Wladimir J. van der Laan	bdc2644b72	Merge #18107: build: Add cov_fuzz target ... faf7d4fa86 build: Add cov_fuzz target (MarcoFalke) fac71e364e build: link fuzz/test_runner.py for out-of-tree builds (MarcoFalke) faf2c5aca0 build: Remove unused USE_COVERAGE (MarcoFalke) Pull request description: Only libFuzzer is supported right now, so clang is required. Thus, this needs a workaround such as https://github.com/bitcoin/bitcoin/issues/12602#issuecomment-562788247 Can be tested with: ``` mkdir build && cd build ../configure --enable-fuzz --with-sanitizers=fuzzer --enable-lcov --enable-lcov-branch-coverage CC=clang CXX=clang++ make $MAKEJOBS make cov_fuzz ACKs for top commit: practicalswift: ACK faf7d4fa86 Tree-SHA512: 6828f8f81d95f6781713d0b09d7eba2ffdb50217e09ca839db61791a4ed70024859c7a0cb01d9eede79166d574dd57ece01f9d9fe2610d4a72a4ca4a4ce0b838	5 years ago
fanquake	bd37f2bc26	build: remove Boost Chrono detection from build system	5 years ago
fanquake	97aadf98d0	Merge #16117: util: Replace boost sleep with std sleep ... fae86c38bc util: Remove unused MilliSleep (MarcoFalke) fa9af06d91 scripted-diff: Replace MilliSleep with UninterruptibleSleep (MarcoFalke) fa4620be78 util: Add UnintrruptibleSleep (MarcoFalke) Pull request description: We don't use the interruptible feature of boost's sleep anywhere, so replace it with the sleep in `std::thread` ACKs for top commit: ajtowns: ACK fae86c38bc quick code review practicalswift: ACK fae86c38bc -- patch looks correct sipa: Concept and code review ACK fae86c38bc fanquake: ACK fae86c38bc - note that an instance of `DHAVE_WORKING_BOOST_SLEEP_FOR` was missed in the [linter](https://github.com/bitcoin/bitcoin/blob/master/test/lint/extended-lint-cppcheck.sh#L69), but that can be cleaned up later. Tree-SHA512: 7c0f8eb197664b9f7d9fe6c472c77d384f11c797c913afc31de4b532e3b4fd9ea6dd174f92062ff9d1ec39b25e0900ca7c597435add87f0f2477d9557204848c	5 years ago
Dan Gershony	7644567758	Add missing step in win deployment instructions ... As explained in #17864 there is a missing step that was required to finish the compilation for bitcoin core on windows	5 years ago
MarcoFalke	fae86c38bc	util: Remove unused MilliSleep	5 years ago
Hennadii Stepanov	dcb7af053d	Ignore only auto-generated .vcxproj files	5 years ago
MarcoFalke	faf2c5aca0	build: Remove unused USE_COVERAGE	5 years ago
Wladimir J. van der Laan	22d11187ee	Merge #17398: build: Update leveldb to 1.22+ ... 677fb8e923 test: Add ubsan surpression for crc32c (Wladimir J. van der Laan) 8e68bb1dde build: Disable msvc warning 4722 for leveldb build (Aaron Clauson) be23949765 build: MSVC changes for leveldb update (Aaron Clauson) 9ebdf04757 build: CRC32C build system integration (Wladimir J. van der Laan) 402252a808 build: Add LCOV exception for crc32c (Wladimir J. van der Laan) 3a037d0067 test: Add crc32c exception to various linters and generation scripts (Wladimir J. van der Laan) 84ff1b2076 test: Add crc32c to subtree check linter (Wladimir J. van der Laan) 7cf13a5134 doc: Add crc32c subtree to developer notes (Wladimir J. van der Laan) 24d02a9ac0 build: Update build system for new leveldb (Wladimir J. van der Laan) 2e1819311a Squashed 'src/crc32c/' content from commit 224988680f7673cd7c769963d4035cb315aa3388 (Wladimir J. van der Laan) 66480821b3 Squashed 'src/leveldb/' changes from f545dfabff4c2e9836efed094dba99a34fbc6b88..f8ae182c1e5176d12e816fb2217ae33a5472fdd7 (Wladimir J. van der Laan) Pull request description: This updates leveldb to currently newest upstream commit 0c40829872: - CRC32C hardware acceleration is now an external library [crc32c](https://github.com/google/crc32c). This adds acceleration on ARM, and should be faster on x86 because of using prefetch. It also makes it easy to support similar instruction sets on other platforms in the future. - Thread handling uses C++11, instead of platform specific code. - Native windows environment was added. No need to maintain our own hacky one, anymore. - Upstream now builds using CMake. This doesn't mean we need to use that (phew), but internal configuration changed to a a series of checks, instead of OS profiles. This means the blanket error "Cannot build leveldb for $host. Please file a bug report' is removed. All changes: a53934a3ae...0c40829872 Pretty much all our changes have been subsumed by upstream, so we figured it was cleaner to start over with a new branch from upstream with the still-relevant patches applied: https://github.com/bitcoin-core/leveldb/tree/bitcoin-fork-new There's quite some testing to be done (see below). See https://github.com/bitcoin-core/leveldb/issues/25 and https://github.com/bitcoin-core/leveldb/pull/26 for more history and context. TODO: - [x] Subtree `crc32c` - [x] Make linters happy about crc32 subtree - [x] Integrate `crc32c` library into build system - [x] MSVC build system ACKs for top commit: sipa: ACK 677fb8e923 Tree-SHA512: 37ee92a750e053e924bc4626b12bb3fd81faa9f8c5ebaa343931fee810c45ba05aa6051fdea82535fa351bf2be7297801b98af9469865fc5ead771650a5d6240	5 years ago
MarcoFalke	fa1a46e7f4	build: Fix appveyor test_bitcoin build of *.raw	5 years ago
Aaron Clauson	8e68bb1dde	build: Disable msvc warning 4722 for leveldb build ... This prevents AppVeyor from failing on a warning in leveldb's new Windows environment.	5 years ago
Aaron Clauson	be23949765	build: MSVC changes for leveldb update	5 years ago
fanquake	2525c096b0	build: remove configure checks for win libraries we don't link against ... While cross compiling, HOST=x86_64-w64-mingw32, none of these libs actually seem to be passed to the linker.	5 years ago
Wladimir J. van der Laan	7a311fa54a	Merge #17738: build: remove linking librt for backwards compatibility ... f7453dcc03 build: remove linking librt for backwards compatibility (fanquake) Pull request description: Now that we require glibc 2.17+, see #17538, we can remove linking librt for backwards compatibility purposes. The `clock_*` functions from librt were merged into glibc as part of the [2.17 release](https://sourceware.org/ml/libc-announce/2012/msg00001.html): * The `clock_*` suite of functions (declared in <time.h>) is now available directly in the main C library. Previously it was necessary to link with -lrt to use these functions. This change has the effect that a single-threaded program that uses a function such as `clock_gettime' (and is not linked with -lrt) will no longer implicitly load the pthreads library at runtime and so will not suffer the overheads associated with multi-thread support in other code such as the C++ runtime library. Note that `librt` is already unused by the RISC-V and AARCH64 binaries as their librts don't export any `clock_*` functions. As an example, you can find a diff of the arm32 vs arm64 librt symbols [here](https://gist.github.com/fanquake/b08cb1f0d14df3133395d7796ebf030c). Below is the library usage for the `v0.19.0.1` release (can delete these tables pre-merge). #### RISC-V ```bash riscv/bin/bitcoin-cli: ['libpthread.so.0', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-riscv64-lp64d.so.1'] riscv/bin/bitcoin-qt: ['libpthread.so.0', 'libfontconfig.so.1', 'libfreetype.so.6', 'libxcb.so.1', 'libdl.so.2', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-riscv64-lp64d.so.1', 'libatomic.so.1'] riscv/bin/bitcoin-wallet: ['libpthread.so.0', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-riscv64-lp64d.so.1', 'libatomic.so.1'] riscv/bin/bitcoind: ['libpthread.so.0', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-riscv64-lp64d.so.1', 'libatomic.so.1'] riscv/bin/bitcoin-tx: ['libpthread.so.0', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-riscv64-lp64d.so.1'] riscv/bin/test_bitcoin: ['libpthread.so.0', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-riscv64-lp64d.so.1', 'libatomic.so.1'] ``` #### AARCH64 ```bash aarch64/bin/bitcoin-cli: ['libpthread.so.0', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-aarch64.so.1'] aarch64/bin/bitcoin-qt: ['libpthread.so.0', 'libfontconfig.so.1', 'libfreetype.so.6', 'libxcb.so.1', 'libdl.so.2', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-aarch64.so.1'] aarch64/bin/bitcoin-wallet: ['libpthread.so.0', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-aarch64.so.1'] aarch64/bin/bitcoind: ['libpthread.so.0', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-aarch64.so.1'] aarch64/bin/bitcoin-tx: ['libpthread.so.0', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-aarch64.so.1'] aarch64/bin/test_bitcoin: ['libpthread.so.0', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-aarch64.so.1'] ``` #### ARM LINUX GNUEABIHF ```bash arm32/bin/bitcoin-cli: ['libpthread.so.0', 'librt.so.1', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-armhf.so.3'] arm32/bin/bitcoin-qt: ['libpthread.so.0', 'librt.so.1', 'libfontconfig.so.1', 'libfreetype.so.6', 'libxcb.so.1', 'libdl.so.2', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-armhf.so.3'] arm32/bin/bitcoin-wallet: ['libpthread.so.0', 'librt.so.1', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-armhf.so.3'] arm32/bin/bitcoind: ['libpthread.so.0', 'librt.so.1', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-armhf.so.3'] arm32/bin/bitcoin-tx: ['libpthread.so.0', 'librt.so.1', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-armhf.so.3'] arm32/bin/test_bitcoin: ['libpthread.so.0', 'librt.so.1', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-armhf.so.3'] ``` #### LINUX X86_64 ```bash x86_64/bin/bitcoin-cli: ['libpthread.so.0', 'librt.so.1', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-x86-64.so.2'] x86_64/bin/bitcoin-qt: ['libpthread.so.0', 'librt.so.1', 'libfontconfig.so.1', 'libfreetype.so.6', 'libxcb.so.1', 'libdl.so.2', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-x86-64.so.2'] x86_64/bin/bitcoin-wallet: ['libpthread.so.0', 'librt.so.1', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-x86-64.so.2'] x86_64/bin/bitcoind: ['libpthread.so.0', 'librt.so.1', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-x86-64.so.2'] x86_64/bin/bitcoin-tx: ['libpthread.so.0', 'librt.so.1', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-x86-64.so.2'] x86_64/bin/test_bitcoin: ['libpthread.so.0', 'librt.so.1', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux-x86-64.so.2'] ``` #### LINUX i686 ```bash i686/bin/bitcoin-cli: ['libpthread.so.0', 'librt.so.1', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux.so.2'] i686/bin/bitcoin-qt: ['libpthread.so.0', 'librt.so.1', 'libfontconfig.so.1', 'libfreetype.so.6', 'libxcb.so.1', 'libdl.so.2', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux.so.2'] i686/bin/bitcoin-wallet: ['libpthread.so.0', 'librt.so.1', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux.so.2'] i686/bin/bitcoind: ['libpthread.so.0', 'librt.so.1', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux.so.2'] i686/bin/bitcoin-tx: ['libpthread.so.0', 'librt.so.1', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux.so.2'] i686/bin/test_bitcoin: ['libpthread.so.0', 'librt.so.1', 'libm.so.6', 'libgcc_s.so.1', 'libc.so.6', 'ld-linux.so.2'] ``` ACKs for top commit: laanwj: ACK f7453dcc03 Tree-SHA512: b418260edcda88583abfa386a592ebfb977d111e8e2ba887a30bf830b0b10dba429b9cfd615fad453ff0bb824225914ccb91433064b158ae1fbb9d20fc0b9937	5 years ago
MarcoFalke	95ca6aeec7	Merge #17691: doc: Add missed copyright headers ... fac86ac7b3 scripted-diff: Add missed copyright headers (Hennadii Stepanov) 6fde9d5e47 script: Update EXLUDE list in copyright_header.py (Hennadii Stepanov) 1998152f15 script: Add empty line after C++ copyright (Hennadii Stepanov) 071f2fc204 script: Add ability to insert copyright to *.sh (Hennadii Stepanov) Pull request description: This PR improves `contrib/devtools/copyright_header.py` script and adds copyright headers to the files in `src` and `test` directories with two exceptions: - [`src/reverse_iterator.h`](https://github.com/bitcoin/bitcoin/blob/master/src/reverse_iterator.h) (added to exceptions) - [`src/test/fuzz/FuzzedDataProvider.h`](https://github.com/bitcoin/bitcoin/blob/master/src/test/fuzz/FuzzedDataProvider.h) (added to exceptions) On master 5622d8f3156a293e61d0964c33d4b21d8c9fd5e0: ``` $ ./contrib/devtools/copyright_header.py report . | grep zero 25 with zero copyrights ``` With this PR: ``` $ ./contrib/devtools/copyright_header.py report . | grep zero 2 with zero copyrights ``` ~I am uncertain about our copyright policy with `build_msvc` and `contrib` directories content, so they are out of scope of this PR.~ ACKs for top commit: MarcoFalke: ACK fac86ac7b3 Tree-SHA512: d7832c4a7a1a3b7806119775b40ec35d7982f49ff0e6199b8cee4c0e0a36e68d51728b6ee9924b1c161df4bc6105bd93391b79d42914357fa522f499cb113fa8	5 years ago
Aaron Clauson	0874a109da	Ignore msvc linker warning and update to msvc build instructions. ... - Update Visual Studio instructions. - Remove x64 platform conditional from bitcoin-qt project configuration. - Set use native environment toolset to fix linker warning. - Ignore linker warning about precompiled type information missing for test_bitcoin_qt.	5 years ago
Hennadii Stepanov	fac86ac7b3	scripted-diff: Add missed copyright headers ... -BEGIN VERIFY SCRIPT- s() { contrib/devtools/copyright_header.py insert "$1"; } s build_msvc/bitcoin_config.h s build_msvc/msvc-autogen.py s build_msvc/testconsensus/testconsensus.cpp s contrib/devtools/circular-dependencies.py s contrib/devtools/gen-manpages.sh s contrib/filter-lcov.py s contrib/gitian-build.py s contrib/install_db4.sh s src/crypto/sha256_avx2.cpp s src/crypto/sha256_sse41.cpp s src/fs.cpp s src/qt/test/addressbooktests.cpp s src/qt/test/addressbooktests.h s src/qt/test/util.cpp s src/qt/test/util.h s src/qt/test/wallettests.cpp s src/qt/test/wallettests.h s src/test/blockchain_tests.cpp s test/functional/combine_logs.py s test/lint/lint-locale-dependence.sh sed -i '1G' test/lint/lint-shebang.sh s test/lint/lint-shebang.sh -END VERIFY SCRIPT-	5 years ago
fanquake	f7453dcc03	build: remove linking librt for backwards compatibility ... Now that we require glibc 2.17+, #17538, we can remove linking in librt for backwards compatibility purposes. The clock_* functions from librt were merged into glibc as part of the 2.17 release. * The `clock_*' suite of functions (declared in <time.h>) is now available directly in the main C library. Previously it was necessary to link with -lrt to use these functions. This change has the effect that a single-threaded program that uses a function such as `clock_gettime' (and is not linked with -lrt) will no longer implicitly load the pthreads library at runtime and so will not suffer the overheads associated with multi-thread support in other code such as the C++ runtime library. https://sourceware.org/ml/libc-announce/2012/msg00001.html Note that librt is already not linked by the RISC-V and AARCH64 binaries.	5 years ago
Aaron Clauson	592af5ad3a	Moved the include of the system projects to before the build depends on task. Otherwise it doesn't get run.	5 years ago
Aaron Clauson	6e2215187e	Included test_bitcoin-qt in msvc build.	5 years ago
Aaron Clauson	75d9317bc1	Update msvc build for Visual Studio 2019 v16.4 ... msvc warning C4834 for the Bitcoin Core build was introduced by Visual Studio 16.4.0. This PR adds an ignore rule for the warning (it's related to the nodiscard attribute and is not considered relevant). An additional side effect of the msvc compiler update is the prebuilt Qt5.9.8 libraries cannot be linked due to being built with an earlier version of the compiler. To fix this a new Qt5.9.8 version has been compiled and the appveyor job updated to use them. The GitHub Actions job needs to continue to use the original Qt5.9.8 libraries until the latest GitHub Windows image also updates to >= Visual Studio 2019 v16.4.	5 years ago
Aaron Clauson	f13e274b6f	Appveyor install libevent[thread] vcpkg ... As per #17586 the default libevent vcpkg install now has thread disabled. This PR installs libevent with the thread feature enabled.	5 years ago
Aaron Clauson	b509554433	Added libtest_util library to msvc build configuration.	5 years ago
fanquake	397dbae070	ci: remove OpenSSL installation	5 years ago
fanquake	a4eb839619	doc: remove OpenSSL from build instructions and licensing info	5 years ago
Aaron Clauson	29eb039252	Moves vcpkg list to a text file and updates the appveyor job and readme to use it.	5 years ago
Aaron Clauson	b80f7db832	Remove redundant class file includes from test_bitcoin project.	5 years ago
MarcoFalke	46e0e27639	Merge #17364: Updates to appveyor config for VS2019 and Qt5.9.8 + msvc project fixes ... 3c84deebaa Updated appveyor config: - Update build image from Visual Studio 2017 to Visual Studio 2019. - Updated Qt static library from Qt5.9.7 to Qt5.9.8. - Added commands to update vcpkg port files (this does not update already installed packages). - Updated vcpkg package list as per #17309. - Removed commands setting common project file options. Now done via common.init.vcxproj include. - Changed msbuild verbosity from normal to quiet. Normal rights a LOT of logs and impacts appveyor job duration. Updated msvc project configs: - Updated platform toolset from v141 to v142. - Updated Qt static library from Qt5.9.7 to Qt5.9.8. - Added ignore for linker warning building bitcoin-qt program. - Added missing util/str.cpp class file to test_bitcoin project file. (Aaron Clauson) Pull request description: Updates to appveyor config: - Update build image from Visual Studio 2017 to Visual Studio 2019. - Updated Qt static library from Qt5.9.7 to Qt5.9.8. - Added commands to update vcpkg port files (this does not update already installed packages). - Updated vcpkg package list as per #17309. - Removed commands setting common project file options. Now done via common.init.vcxproj include. - Changed msbuild verbosity from normal to quiet. Normal writes a LOT of logs and impacts appveyor job duration. Updates to msvc project configs: - Updated platform toolset from v141 to v142. - Updated Qt static library from Qt5.9.7 to Qt5.9.8. - Added ignore for linker warning building bitcoin-qt program. - Added missing util/str.cpp class file to test_bitcoin project file. In order for an existing appveyor job based on the new config to work the cache must be purged. The steps to do this are shown below. The specific appveyor project path will need to be adjusted. ```` export APPVEYOR_TOKEN="<your-api-token>" curl -H "Authorization: Bearer $APPVEYOR_TOKEN" -X DELETE https://ci.appveyor.com/api/projects/bitcoin/bitcoin-9ql6k/buildcache ```` ACKs for top commit: ryanofsky: Non-expert code review ACK 3c84deebaa. Tree-SHA512: 77063d4588c3499de78b0bcc4d8b638f36c70284485ae94ce5c718a3dacb6d28cc34f9443c54c4e98c07b446d26b59589259671c2f6bcc952344042b4a3baf8f	5 years ago
Aaron Clauson	3c84deebaa	Updated appveyor config: ... - Update build image from Visual Studio 2017 to Visual Studio 2019. - Updated Qt static library from Qt5.9.7 to Qt5.9.8. - Added commands to update vcpkg port files (this does not update already installed packages). - Updated vcpkg package list as per #17309. - Removed commands setting common project file options. Now done via common.init.vcxproj include. - Changed msbuild verbosity from normal to quiet. Normal rights a LOT of logs and impacts appveyor job duration. Updated msvc project configs: - Updated platform toolset from v141 to v142. - Updated Qt static library from Qt5.9.7 to Qt5.9.8. - Added ignore for linker warning building bitcoin-qt program. - Added missing util/str.cpp class file to test_bitcoin project file.	5 years ago
MarcoFalke	faec28252c	scripted-diff: test: Move setup_common to test library ... -BEGIN VERIFY SCRIPT- # Move files for f in $(git ls-files src/test/lib/); do git mv $f src/test/util/; done git mv src/test/setup_common.cpp src/test/util/ git mv src/test/setup_common.h src/test/util/ # Replace Windows paths sed -i -e 's|\\setup_common|\\util\\setup_common|g' $(git grep -l '\\setup_common') sed -i -e 's|src\\test\\lib\\|src\\test\\util\\|g' build_msvc/test_bitcoin/test_bitcoin.vcxproj # Everything else sed -i -e 's|/setup_common|/util/setup_common|g' $(git grep -l 'setup_common') sed -i -e 's|test/lib/|test/util/|g' $(git grep -l 'test/lib/') # Fix include guard sed -i -e 's|BITCOIN_TEST_SETUP_COMMON_H|BITCOIN_TEST_UTIL_SETUP_COMMON_H|g' ./src/test/util/setup_common.h sed -i -e 's|BITCOIN_TEST_LIB_|BITCOIN_TEST_UTIL_|g' $(git grep -l 'BITCOIN_TEST_LIB_') -END VERIFY SCRIPT-	5 years ago
fanquake	b1f1fb5f1d	doc: update MSVC instructions to remove Qt configuration ... Also adds missing Boost packages. Installing only the currently listed packages was not sufficient to complete a build.	5 years ago
fanquake	befbc40eb5	build: remove EVP_MD_CTX_new detection ... This was added in #9475 to fix LibreSSL compatibility for BIP70, so is no longer required.	5 years ago
Sebastian Falbesoner	a0fc076476	refactor: test/bench: dedup Build{Crediting,Spending}Transaction() ... prototypes used in src/test/script_tests.cpp: - CMutableTransaction BuildCreditingTransaction(const CScript& scriptPubKey, int nValue = 0); - CMutableTransaction BuildSpendingTransaction(const CScript& scriptSig, const CScriptWitness& scriptWitness, const CTransaction& txCredit); prototypes used in bench/verify_script.cpp: - CMutableTransaction BuildCreditingTransaction(const CScript& scriptPubKey); - CMutableTransaction BuildSpendingTransaction(const CScript& scriptSig, const CMutableTransaction& txCredit); The more generic versions from the script tests are moved into a new file pair transaction_utils.cpp/h and the calls are adapted accordingly in the verify_script benchmark (passing the nValue of 1 explicitely for BuildCreditingTransaction(), passing empty scriptWitness explicitely and converting txCredit parameter to CTransaction in BuildSpendingTransaction()).	5 years ago
Jon Atack	c0859b7dac	build: 0.19 release updates on master ... as per doc/release-process.md. Note: On branch-off, these same changes should be made on the release branch, with also these additional changes to both files: - set `CLIENT_VERSION_REVISION` to `0` - set `CLIENT_VERSION_IS_RELEASE` to `true`	5 years ago
Aaron Clauson	1619684322	Added libbitcoin_qt and bitcoin-qt to the msbuild configuration.	5 years ago
Aaron Clauson	d53d591230	Added the bench_bitcoin project to the list automatically produced by the msvc-autogen python script.	5 years ago
Aaron Clauson	aff1c9c884	Adds an option to msbuild common configuration to ignore linker warning 4221. This warning is for object files that do not include any symbols. The warning is harmless and occurs due to some classes that are *nix only having no source to compile for an msvc build.	5 years ago
Wladimir J. van der Laan	390874c722	qt: Remove menu icons	5 years ago
Aaron Clauson	e0324c3768	Updated python command in readme so it will work on systems that have both python2 and 3 installed.	5 years ago
Wladimir J. van der Laan	0a6ee9797e	Merge #16267: bench: Benchmark blockToJSON ... 91509ffe24 bench: Benchmark blockToJSON (Kirill Fomichev) Pull request description: Related: - "getblock performance issue on verbosity" https://github.com/bitcoin/bitcoin/issues/15925 - "refactor: Avoid UniValue copy constructor" #15974 ACKs for top commit: laanwj: ACK 91509ffe24 Tree-SHA512: e70b12cb31921c7527bde334f52f39776da698b6bbdb196079a8b68478c67585a5bd7bed7403f65166bd604f7ed60778c53dc064d743bb8368318a1283d1073e	5 years ago
Wladimir J. van der Laan	8c69fae944	Merge #15457: Check std::system for -[alert|block|wallet]notify ... f874e14cd3 [build]: check std::system for -[alert|block|wallet]notify (Sjors Provoost) cc3ad56ff2 [build] MSVC: set HAVE_SYSTEM for desktop apps (Sjors Provoost) c1c91bb78d [build] detect std::system or ::wsystem (Sjors Provoost) Pull request description: Platforms such as iOs and Universal Windows Platform do not support launching a process through system(). ACKs for top commit: laanwj: code review ACK f874e14cd3 Tree-SHA512: 16bb4a8fa1896046ccb22a46c8985e1aa45f5b11ecf5539eb2299e9a58f1a5b085c0c12cb6939c7493d93abce7e84fadcbfc73374c887db63da6d00c08aa476d	5 years ago
Kirill Fomichev	91509ffe24	bench: Benchmark blockToJSON	5 years ago
fanquake	4f378ac30c	Merge #16308: [MSVC] Copy build output to src/ automatically after build ... 08c721dab2 [MSVC] Copy build output to src/ automatically after build (nicolas.dorier) Pull request description: I got frustrated many time that I run functional tests on an outdated binary. This make sure that files are properly copied (and debuggable) for functional tests after a build. This PR add a `common.vcxproj` which can be used to extend the build steps. I can't move it in `common.init`, because build step customization must always come at the end of a project file. ACKs for top commit: sipsorcery: tACK 08c721dab2. fanquake: ACK 08c721dab2 Tree-SHA512: 7c033552a7bbbd3bb51a72d13bf4254e3b779c59ec93629c85f17745a1b17a8c1d6c87bb5e50eb5f4e5486e767929c0126e555b185608fd666aa39729b5068e1	5 years ago
MarcoFalke	91c345eb92	Merge #16299: bench: Move generated data to a dedicated translation unit ... 3d60a03a7c bench: Move generated data to a dedicated translation unit (João Barbosa) Pull request description: With this change multiple benchmarks can use the same data without incurring in a bigger binary. ACKs for top commit: laanwj: code review ACK 3d60a03a7c Tree-SHA512: 8903bb09e4327c88e585a09bc7df1cbdfc18ebdc5d9c86bf3d6d9252a05eaf18b14ecd2bafdacd82f05a659e4b35ecd301c36011c97f7bf89302793165b00fdc	5 years ago
João Barbosa	3d60a03a7c	bench: Move generated data to a dedicated translation unit	5 years ago
nicolas.dorier	08c721dab2	[MSVC] Copy build output to src/ automatically after build	5 years ago
nicolas.dorier	c83f0ac9b2	[MSVC] allow user level project customization	5 years ago
nicolas.dorier	819c5ddad3	[MSVC] Enable Fuzz for functional tests	5 years ago
nicolas.dorier	e47e79377f	[MSVC]: Create the config.ini as part of bitcoind build	5 years ago
nicolas.dorier	eb832cd3b4	MSVC: Fix error in debug mode (Fix #16245)	5 years ago
MarcoFalke	b1344eac5f	Merge #16241: docs: add rapidcheck to vcpkg install list ... 4971be76a7 docs: add rapidcheck to vcpkg install list (fanquake) Pull request description: Setting up a fresh Windows 10 VM using [this guide](https://github.com/fanquake/core-review/blob/master/windows.md), installing `rapidcheck` and building using the changes in #16235 resulted in a working `bitcoind` and all tests passing. ACKs for commit 4971be: Tree-SHA512: d0cb2d1d6ac5cdecf77c21f9b05e8803266511dbd06cb644352a229d101c7bf63f8022822852877371cce83c414275a850eb9ab6876a61c2fd1061627b7223f7	5 years ago
fanquake	4971be76a7	docs: add rapidcheck to vcpkg install list	5 years ago
Aaron Clauson	750d1bc7df	Cleaned up and consolidated msvc build files to allow faster builds and easier migration to vs2019.	5 years ago
Sjors Provoost	cc3ad56ff2	[build] MSVC: set HAVE_SYSTEM for desktop apps	6 years ago
Jon Atack	3ee28c506d	build: bump bitcoin_config.h packages to v0.18 ... Follow-up to 48ed65b.	6 years ago
Hennadii Stepanov	48ed65bcdd	Align MSVC build options with Linux build ones	6 years ago
MarcoFalke	fa46ac3127	bench: Add wallet_balance benchmarks	6 years ago
MarcoFalke	fa821904bf	scripted-diff: Rename test_bitcoin to test/setup_common ... -BEGIN VERIFY SCRIPT- sed -i --regexp-extended -e 's/test_bitcoin\.(h|cpp)/setup_common.\1/g' $(git grep -l test_bitcoin) git mv ./src/test/test_bitcoin.h ./src/test/setup_common.h git mv ./src/test/test_bitcoin.cpp ./src/test/setup_common.cpp sed -i -e 's/BITCOIN_TEST_TEST_BITCOIN_H/BITCOIN_TEST_SETUP_COMMON_H/g' ./src/test/setup_common.h -END VERIFY SCRIPT-	6 years ago
MarcoFalke	fa8685d49e	test: Use test_bitcoin setup in bench, Add test utils	6 years ago
John Newbery	9eaeb7fb8d	[build] Move wallet load functions to wallet/load unit ... Moves the following wallet load functions to a new wallet/load unit in the libbitcoin_wallet library. All other functions in wallet/init remain in libbitcoin_server: - `VerifyWallets` - `LoadWallets` - `StartWallets` - `FlushWallets` - `StopWallets` - `UnloadWallets`	6 years ago
John Newbery	fdf8888b6f	[build] Move CheckTransaction from lib_server to lib_consensus ... CheckTransaction is a context-free function that does not require access to the blockchain or mempool. Move it from src/consensus/tx_verify in lib_server to a new unit src/consensus/tx_check in lib_consensus so that it can be called by non-server libraries.	6 years ago