diff options
| author | fanquake <fanquake@gmail.com> | 2019-09-18 08:42:10 +0800 |
|---|---|---|
| committer | fanquake <fanquake@gmail.com> | 2019-09-18 10:26:33 +0800 |
| commit | cc1d7fd57c9e777f7fb6da6a488b18317fbcd2c2 (patch) | |
| tree | 025dceadc9ea0844ed57d9f821b6dd71263a09d9 /src/consensus | |
| parent | b3067f4338d901394fb0237567cb5e19ccee5f8a (diff) | |
| parent | 93995c27515aa268c2c1aefe82f1a3cee33966ce (diff) | |
| download | bitcoin-cc1d7fd57c9e777f7fb6da6a488b18317fbcd2c2.tar.xz | |
Merge #16879: build: remove redundant sed patching
93995c27515aa268c2c1aefe82f1a3cee33966ce build: remove unnecessary qt xcb patching (fanquake)
4d45577c4314fd7075094d5b3384b4b86100c125 build: remove unnecessary macOS qt patching (fanquake)
Pull request description:
While looking at #16838 I found at least two cases of `sed` patching in depends that now seems to be redundant. There's possibly a [third case](https://github.com/bitcoin/bitcoin/pull/16837#discussion_r322842701), but I haven't looked into that enough yet.
Patching `0` -> `kCGMouseButtonLeft` should not be required, as [`kCGMouseButtonLeft`](https://code.qt.io/cgit/qt/qtbase.git/tree/src/plugins/platforms/cocoa/qcocoacursor.mm?h=5.9#n82) has been used in the `cocoa/qcocoacursor.mm` source for a while.
The include we were modifying in [`src/plugins/platforms/xcb/qxcbxsettings.cpp`](https://code.qt.io/cgit/qt/qtbase.git/tree/src/plugins/platforms/xcb/qxcbxsettings.cpp?h=5.9#n47) was removed in [this commit](https://code.qt.io/cgit/qt/qtbase.git/commit/src/plugins/platforms/xcb/qxcbxsettings.cpp?h=5.9&id=78731b434e0e99ad108601249108e12d8a49c350).
ACKs for top commit:
dongcarl:
ACK 93995c27515aa268c2c1aefe82f1a3cee33966ce
Tree-SHA512: 5e0cbf317b798ce2e142a42b7fabf1d9e8e00d12f59589e98d790195ba27db60858f933b035c6e9cd0deadd8c3406f1ff4a4ed2af4a19e9b5b43aa97d04b9ecb
Diffstat (limited to 'src/consensus')
0 files changed, 0 insertions, 0 deletions