diff options
| author | fanquake <fanquake@gmail.com> | 2021-05-02 12:57:04 +0800 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2021-05-02 13:04:02 +0800 |
| commit | f2865b739417ca752dbbbd24828b7b2628394c19 (patch) | |
| tree | 48528e6b19726af4f8e9b0b4c6c47a9e5e6f49c4 /.github | |
| parent | d22e7ee93313b13365bd14a5fffeb055cff4dcd2 (diff) | |
| parent | 0a331456e44ce2bdd3bfe62f7603a312905c6624 (diff) | |
Merge bitcoin/bitcoin#21811: doc: Remove Visual Studio 2017 reference from readme
0a331456e44ce2bdd3bfe62f7603a312905c6624 Remove Visual Studio 2017 reference from readme (Aaron Clauson)
Pull request description:
This PR was motivated by a comment in GUI PR (257) regarding a suggested improvement not being supported by VS2017.
When checking whether master can still be built with the VS2017 toolset ABI issues were encountered. Most likely due to the pre-compiled Qt binaries that are used.
It does not seem worth the effort to try and support VS2017, which would most likely require additional Qt binaries, or lengthy instructions on how to build static Qt binaries on Windows (which is very error prone and tedious).
ACKs for top commit:
jarolrod:
ACK 0a331456e44ce2bdd3bfe62f7603a312905c6624
hebasto:
ACK 0a331456e44ce2bdd3bfe62f7603a312905c6624, I have reviewed the code and it looks OK, I agree it can be merged.
Tree-SHA512: 17e2326cd55a5cc3fc13622ba72bb44f9b9d55070cb77941df5fd943cd3f1baf02b9ac9504dfae5941c522748ad7b584c3c8da03fb323a87b3639eb926ce1699
Diffstat (limited to '.github')
0 files changed, 0 insertions, 0 deletions