diff options
author | fanquake <fanquake@gmail.com> | 2019-09-11 16:18:32 +1000 |
---|---|---|
committer | fanquake <fanquake@gmail.com> | 2019-09-11 16:36:03 +1000 |
commit | 2324aa1dc409e9496b7083aaef5fcb20849f33c0 (patch) | |
tree | b7b749090c3ae45f2cfe4bf6e6614cbce66dc3a2 /INSTALL.md | |
parent | 2296fe65f598eefb72e944cefcca462e740bfaf3 (diff) | |
parent | 161968432205a5bdf9a98b99562e956be8c0db89 (diff) | |
download | bitcoin-2324aa1dc409e9496b7083aaef5fcb20849f33c0.tar.xz |
Merge #15529: Add Qt programs to msvc build (updated, no code changes)
161968432205a5bdf9a98b99562e956be8c0db89 Added libbitcoin_qt and bitcoin-qt to the msbuild configuration. (Aaron Clauson)
Pull request description:
This PR has ~~90%~~ all of the work done to allow the bitcoin Qt programs to be built with msvc and the appveyor script.
Outstanding issues:
- ~~There are ~~3~~ ~~6~~ 5 code tweaks required for the bitcoin Qt components to be built without warnings with msvc. They seem minor~~,
- Building Qt as a static library for Windows is painful and time consuming. I doubt it will ever be possible to build Qt from source as part of an appveyor job (and it would probably take over an hour even if it was). My tentative solution is to build locally and upload the binaries as a [github release](https://github.com/sipsorcery/qt_win_binary/releases). The msvc build is only for testing and tinkering but even so this doesn't feel like the ideal solution. Open to suggestions?
- ~~There is still an issue to sort out with the payment request URL handling. Building Qt with openssl is an extra headache. I will continue to work on getting this working.~~
The big benefit of this PR is the ability to run bitcoin-qt within a Visual Studio debugging session which could expedite tracking down issues on Windows.
On a side note the test-bitcoin-qt tests fail very early, probably due to *nix specific tests. I haven't dug into them at this point.
**Update 28 Jun 2019**: The ENABLE_BIP70 option is now off (it's flagged for removal as per #15584). With it disabled msbuild does not require any code changes to build the Bitcoin Core Qt applications.
ACKs for top commit:
fanquake:
re-ACK 161968432205a5bdf9a98b99562e956be8c0db89 - AppVeyor looks ok now.
Tree-SHA512: c0d3fd53b3ff99096b2505d519ed5ca6791bc4bce77addf9c520dc042eec5980a51a1fb9f0aa72e9cc53773085c43218793ca7a915a47806a3a1ffb84d9409f9
Diffstat (limited to 'INSTALL.md')
0 files changed, 0 insertions, 0 deletions