-
Notifications
You must be signed in to change notification settings - Fork 38.6k
build, qt: Fix compiling qt package in depends with GCC 11 #22186
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
|
The following sections might be updated with supplementary metadata relevant to reviewers and maintainers. ConflictsReviewers, this pull request conflicts with the following ones:
If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first. |
|
Guix builds: bash-5.1# find guix-build-$(git rev-parse --short=12 HEAD)/output/ -type f -print0 | env LC_ALL=C sort -z | xargs -r0 sha256sum
56c8da5cb1f260408cf9fc8cc9b3579a8594acb2333f4d00362e756481d7ff1b guix-build-d1d1cc983146/output/aarch64-linux-gnu/bitcoin-d1d1cc983146-aarch64-linux-gnu-debug.tar.gz
bb58ad0641c2e0bd2ea4cd0bc09fb2c02c9fc799e2ea56621f2038bb9e0464a0 guix-build-d1d1cc983146/output/aarch64-linux-gnu/bitcoin-d1d1cc983146-aarch64-linux-gnu.tar.gz
0ff579d862a38bd1104b2ce5a21f85d507e04d06362cd25b41bf2116706bd3c6 guix-build-d1d1cc983146/output/aarch64-linux-gnu/inputs.SHA256SUMS
48e47fa77298d5f3828b468659011e24bef3798cc58a800cb433ebb90a090e78 guix-build-d1d1cc983146/output/arm-linux-gnueabihf/bitcoin-d1d1cc983146-arm-linux-gnueabihf-debug.tar.gz
ecf9e6f2fdd513342ddc92953586ce0957de0ef720f5d667c9fbae74adb465c5 guix-build-d1d1cc983146/output/arm-linux-gnueabihf/bitcoin-d1d1cc983146-arm-linux-gnueabihf.tar.gz
0ff579d862a38bd1104b2ce5a21f85d507e04d06362cd25b41bf2116706bd3c6 guix-build-d1d1cc983146/output/arm-linux-gnueabihf/inputs.SHA256SUMS
e3b0c44298fc1c149afbf4c8996fb92427ae41e4649b934ca495991b7852b855 guix-build-d1d1cc983146/output/dist-archive/SKIPATTEST.TAG
d566fea165cfba3d2bc7cf3e03209464b90ace7b2f4b64c990afcb773117d50c guix-build-d1d1cc983146/output/dist-archive/bitcoin-d1d1cc983146.tar.gz
fd18a17e01691481b98712cb56f80bd8796cabc24c79d976fdcb65546f3be1e4 guix-build-d1d1cc983146/output/powerpc64-linux-gnu/bitcoin-d1d1cc983146-powerpc64-linux-gnu-debug.tar.gz
32cb46abb2e974352cb20decf02205ac07deb3f0769bd821e14ddefde0d5c523 guix-build-d1d1cc983146/output/powerpc64-linux-gnu/bitcoin-d1d1cc983146-powerpc64-linux-gnu.tar.gz
0ff579d862a38bd1104b2ce5a21f85d507e04d06362cd25b41bf2116706bd3c6 guix-build-d1d1cc983146/output/powerpc64-linux-gnu/inputs.SHA256SUMS
a984d9bae9c33dd7f2c0c9eeb8d8509c4bd7c231cd268c04bc71a8c2cb44214e guix-build-d1d1cc983146/output/powerpc64le-linux-gnu/bitcoin-d1d1cc983146-powerpc64le-linux-gnu-debug.tar.gz
e8968956fedba9c6109cbfd40a2af2a5aa6f324f5d3e73c0e54024297a634052 guix-build-d1d1cc983146/output/powerpc64le-linux-gnu/bitcoin-d1d1cc983146-powerpc64le-linux-gnu.tar.gz
0ff579d862a38bd1104b2ce5a21f85d507e04d06362cd25b41bf2116706bd3c6 guix-build-d1d1cc983146/output/powerpc64le-linux-gnu/inputs.SHA256SUMS
e59258b63e14773a902e93a9b56cfe1dcc362fdf1a94489cfc256716ea62e8d3 guix-build-d1d1cc983146/output/riscv64-linux-gnu/bitcoin-d1d1cc983146-riscv64-linux-gnu-debug.tar.gz
2e8ea8e446f5de1ac7cda703a0e1a7c6c45f354cb9f75ca3fdf42e49841a8dbe guix-build-d1d1cc983146/output/riscv64-linux-gnu/bitcoin-d1d1cc983146-riscv64-linux-gnu.tar.gz
0ff579d862a38bd1104b2ce5a21f85d507e04d06362cd25b41bf2116706bd3c6 guix-build-d1d1cc983146/output/riscv64-linux-gnu/inputs.SHA256SUMS
f97cf5498c7dcf41ee9e187dd49c2ebf6046d6e0a9164f0d08452f490b6246c3 guix-build-d1d1cc983146/output/x86_64-apple-darwin18/bitcoin-d1d1cc983146-osx-unsigned.dmg
8e973af8c8408e4c103f159d28288de72f5afad681e83949d822bd87a50d0c48 guix-build-d1d1cc983146/output/x86_64-apple-darwin18/bitcoin-d1d1cc983146-osx-unsigned.tar.gz
2aa752ab55a1dbeed370bceaf816b2a17c0f05f19e237115e65bf146d4e30b84 guix-build-d1d1cc983146/output/x86_64-apple-darwin18/bitcoin-d1d1cc983146-osx64.tar.gz
0ff579d862a38bd1104b2ce5a21f85d507e04d06362cd25b41bf2116706bd3c6 guix-build-d1d1cc983146/output/x86_64-apple-darwin18/inputs.SHA256SUMS
4465de183452ef89a0cdbe9d4c29867fad17cc9df965b8220cb1ca40089fa226 guix-build-d1d1cc983146/output/x86_64-linux-gnu/bitcoin-d1d1cc983146-x86_64-linux-gnu-debug.tar.gz
5871f4436f03f53069e2c6256d19942968d7185658ee285edf1796876553d1fc guix-build-d1d1cc983146/output/x86_64-linux-gnu/bitcoin-d1d1cc983146-x86_64-linux-gnu.tar.gz
0ff579d862a38bd1104b2ce5a21f85d507e04d06362cd25b41bf2116706bd3c6 guix-build-d1d1cc983146/output/x86_64-linux-gnu/inputs.SHA256SUMS
f8c0dd0dd18256133c561763c32184671640b7f4db00fa7920a0a5d66ec099ac guix-build-d1d1cc983146/output/x86_64-w64-mingw32/bitcoin-d1d1cc983146-win-unsigned.tar.gz
e60a3ad3ff5cfca2309ada1952dda7b1e76717e6981d3c222ea3515e46e2aa09 guix-build-d1d1cc983146/output/x86_64-w64-mingw32/bitcoin-d1d1cc983146-win64-debug.zip
f28b1dc6d4d5e115c60f2cf12431902ed4d54241230f0bbe9293b0e3e00c1c9b guix-build-d1d1cc983146/output/x86_64-w64-mingw32/bitcoin-d1d1cc983146-win64-setup-unsigned.exe
3282a1c8bda8522336c19bf574bb0c6093b3a0d5dabe3d88f99d3c62b9d04752 guix-build-d1d1cc983146/output/x86_64-w64-mingw32/bitcoin-d1d1cc983146-win64.zip
0ff579d862a38bd1104b2ce5a21f85d507e04d06362cd25b41bf2116706bd3c6 guix-build-d1d1cc983146/output/x86_64-w64-mingw32/inputs.SHA256SUMS |
|
Gitian builds: # macOS:
21acaab3b9f1b94746e7b05748c64236a86e60709f7470098beafd8541f00a3a bitcoin-d1d1cc983146-osx-unsigned.dmg
a50ab21814742360207b2f89d9056259de8d61d1a7feb1a529d8f762b44aff45 bitcoin-d1d1cc983146-osx-unsigned.tar.gz
69b4d1fa5825e4e0c0e6e43a3dc05ceb870eb3c69eed2bd1718c0543b2977648 bitcoin-d1d1cc983146-osx64.tar.gz
d566fea165cfba3d2bc7cf3e03209464b90ace7b2f4b64c990afcb773117d50c src/bitcoin-d1d1cc983146.tar.gz
fa62d560e307d1565182053183429f8e80f5a910cb2bf5da9d43169ad642eb66 bitcoin-core-osx-22-res.yml
# Windows:
8be1e2e48b461610d40211d9885e89829eb44c1243bc51208f6ba618b2327137 bitcoin-d1d1cc983146-win-unsigned.tar.gz
f7cc5e0ab8a8fbd62a6688af164b44f1a20f1da0628791c41c2ba292524fd7ae bitcoin-d1d1cc983146-win64-debug.zip
08f7181dc91088fa20af4a4018a65b397dd9e3d77ff466322d90d8e21fe85922 bitcoin-d1d1cc983146-win64-setup-unsigned.exe
cdc82a359d77d19bbcc2647d76cdc2ce1f93e8294e478cebd4d22a77ad2a1e8d bitcoin-d1d1cc983146-win64.zip
d566fea165cfba3d2bc7cf3e03209464b90ace7b2f4b64c990afcb773117d50c src/bitcoin-d1d1cc983146.tar.gz
391381b3f6d5c62c8308b12052d3fc5c14b82be7b9ac625cd74690f0f8c6544d bitcoin-core-win-22-res.yml
# Linux:
8b67e9e71701c721529b8623b538acebe0ddb3e66bd63a02145b07407bad3648 bitcoin-d1d1cc983146-aarch64-linux-gnu-debug.tar.gz
b09b988b94af56f30eea69364935547173f6caa43cbd608b24bf8e2dfda87a65 bitcoin-d1d1cc983146-aarch64-linux-gnu.tar.gz
c10592b488eea9e67f805b0ce2ada0216e29186222d7d837851dacf32535731a bitcoin-d1d1cc983146-arm-linux-gnueabihf-debug.tar.gz
262a6143ed63ac1e9645a2df91968d3421cd261468a84eacc3aec774012d5867 bitcoin-d1d1cc983146-arm-linux-gnueabihf.tar.gz
e5d1a7f9683f289270c202f7d7ac771272fb857668c1d9b599dd1219a4691313 bitcoin-d1d1cc983146-powerpc64-linux-gnu-debug.tar.gz
cad1c80cad14293554d2a816e5d891fa94d5adfe844306622fba087999736f7a bitcoin-d1d1cc983146-powerpc64-linux-gnu.tar.gz
18c1a388ee614692e603069055312458618f3f25825596bc9264d7183f933bf3 bitcoin-d1d1cc983146-powerpc64le-linux-gnu-debug.tar.gz
633c242871746686a7900708ce3b8cde3148270b8cecd068e267442240e4a35a bitcoin-d1d1cc983146-powerpc64le-linux-gnu.tar.gz
b61a57080ceb7cef984e8d71cc91a49b7f5948fd05721744bb20fe6a246fe110 bitcoin-d1d1cc983146-riscv64-linux-gnu-debug.tar.gz
807930b58edbc870ceb6607223069f44fc10d23a1d0035a75175e090a6084fae bitcoin-d1d1cc983146-riscv64-linux-gnu.tar.gz
26f8d9080a388015b69454e6c4e5c9fa61f275b3616932012a9953653db25fa6 bitcoin-d1d1cc983146-x86_64-linux-gnu-debug.tar.gz
26aaa250f45ced7edd413de4a236445db34fc7250b1aadb84e413cc2a1a80b8c bitcoin-d1d1cc983146-x86_64-linux-gnu.tar.gz
d566fea165cfba3d2bc7cf3e03209464b90ace7b2f4b64c990afcb773117d50c src/bitcoin-d1d1cc983146.tar.gz
99e3af2dc89391ded8119fac0faf0e2ca2ebd688c93de3dd3c278d0a0bfaa661 bitcoin-core-linux-22-res.yml |
Guix builds: |
Gitian builds:
|
fanquake
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ACK d1d1cc9
…ith GCC 11 d1d1cc9 build, qt: Fix compiling qt package in depends with GCC 11 (Hennadii Stepanov) Pull request description: The `qt` package in depends fails to compile with GCC 11 due to the missed `<limits>` headers. See: https://bugreports.qt.io/browse/QTBUG-90395 Affected systems: - Ubuntu 21.04 + GCC 11.1.0 - Fedora 34 + GCC 11.1.1 Ubuntu 21.04 build log excerpt: ``` Configuring qt... Creating qmake... gmake[1]: Entering directory '/home/hebasto/bitcoin/depends/work/build/x86_64-pc-linux-gnu/qt/5.12.11-1ff5c6c1f55/qtbase/qmake' In file included from /home/hebasto/bitcoin/depends/work/build/x86_64-pc-linux-gnu/qt/5.12.11-1ff5c6c1f55/qtbase/include/QtCore/qendian.h:1, from /home/hebasto/bitcoin/depends/work/build/x86_64-pc-linux-gnu/qt/5.12.11-1ff5c6c1f55/qtbase/src/corelib/codecs/qutfcodec.cpp:43: /home/hebasto/bitcoin/depends/work/build/x86_64-pc-linux-gnu/qt/5.12.11-1ff5c6c1f55/qtbase/include/QtCore/../../src/corelib/global/qendian.h: In static member function ‘static constexpr QSpecialInteger<S> QSpecialInteger<S>::max()’: /home/hebasto/bitcoin/depends/work/build/x86_64-pc-linux-gnu/qt/5.12.11-1ff5c6c1f55/qtbase/include/QtCore/../../src/corelib/global/qendian.h:331:35: error: ‘numeric_limits’ is not a member of ‘std’ 331 | { return QSpecialInteger(std::numeric_limits<T>::max()); } | ^~~~~~~~~~~~~~ /home/hebasto/bitcoin/depends/work/build/x86_64-pc-linux-gnu/qt/5.12.11-1ff5c6c1f55/qtbase/include/QtCore/../../src/corelib/global/qendian.h:331:54: error: ‘::max’ has not been declared; did you mean ‘std::max’? 331 | { return QSpecialInteger(std::numeric_limits<T>::max()); } | ^~~ | std::max ``` ACKs for top commit: fanquake: ACK d1d1cc9 Tree-SHA512: 2dd643efc0aefc492f9565c0900ba0f1657c016bc4a44792f85478b9fc6e0e4ecad78847114ef6ec702d2de4cdbc3f657e9b96634ea58f42b6cc98dfb5e09eab
Gitian builds
|
26ce35f build, qml: Fix compiling with GCC 11 (Hennadii Stepanov) Pull request description: Currently (a8dbc01), building the `qt` package in depends with GCC 11 fails: ``` $ gcc --version gcc (Ubuntu 11.1.0-1ubuntu1~21.04) 11.1.0 Copyright (C) 2021 Free Software Foundation, Inc. This is free software; see the source for copying conditions. There is NO warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. $ make -C depends qt ... In file included from ../../include/QtQml/5.12.11/QtQml/private/qv4propertykey_p.h:1, from ../../include/QtQml/5.12.11/QtQml/private/../../../../../src/qml/jsruntime/qv4internalclass_p.h:56, from ../../include/QtQml/5.12.11/QtQml/private/qv4internalclass_p.h:1, from ../qml/jsruntime/qv4value_p.h:59, from ../qml/jsruntime/qv4managed_p.h:54, from ../qml/jsruntime/qv4engine_p.h:54, from ../qml/jsruntime/qv4engine.cpp:39: ../../include/QtQml/5.12.11/QtQml/private/../../../../../src/qml/jsruntime/qv4propertykey_p.h: In member function ‘uint QV4::PropertyKey::asArrayIndex() const’: ../../include/QtQml/5.12.11/QtQml/private/../../../../../src/qml/jsruntime/qv4propertykey_p.h:116:73: error: ‘numeric_limits’ is not a member of ‘std’ 116 | uint asArrayIndex() const { return (isManaged() || val == 0) ? std::numeric_limits<uint>::max() : static_cast<uint>(val & 0xffffffff); } | ^~~~~~~~~~~~~~ ../../include/QtQml/5.12.11/QtQml/private/../../../../../src/qml/jsruntime/qv4propertykey_p.h:116:92: error: expected primary-expression before ‘>’ token 116 | uint asArrayIndex() const { return (isManaged() || val == 0) ? std::numeric_limits<uint>::max() : static_cast<uint>(val & 0xffffffff); } | ^ ../../include/QtQml/5.12.11/QtQml/private/../../../../../src/qml/jsruntime/qv4propertykey_p.h:116:95: error: ‘::max’ has not been declared; did you mean ‘std::max’? 116 | uint asArrayIndex() const { return (isManaged() || val == 0) ? std::numeric_limits<uint>::max() : static_cast<uint>(val & 0xffffffff); } | ^~~ | std::max ``` This is similar to the bitcoin/bitcoin#22186. ACKs for top commit: promag: utACK 26ce35f. Including `<limits>` should fix error in OP. Tree-SHA512: 1706be13d347e26ebad8029ed5c68719d814b291175afb0de7654a5810f52f8d31d970db329355658dbdc31b58a0c2dc759f3718752ac42d30858f602beac2b6
The
qtpackage in depends fails to compile with GCC 11 due to the missed<limits>headers.See: https://bugreports.qt.io/browse/QTBUG-90395
Affected systems:
Ubuntu 21.04 build log excerpt: