build: Check QT library version #15706

pull lucayepa wants to merge 1 commits into bitcoin:master from lucayepa:check-qt-version changing 1 files +26 −16
  1. lucayepa commented at 2:19 pm on March 30, 2019: contributor

    Fixes #15688

    Due to a bug, the version of the QT library was not checked at configure time for systems using pkg-config. Without any check at configure time, the build process stopped with unexplicative errors on systems with QT versions not supported.

    This PR introduces the control of the version of the QT library, with a warning, or an error, at configure time, if the installed version is not supported.

    Tested in all the following cases (all combinations):

    • on Debian Jessie (QT 5.3.2) and on Debian Stretch (QT 5.7.1)
    • with depends system (QT 5.9.7) enabled and disabled
    • with --with-gui and --without-gui on configure command line

    Since the current 0.18 branch is affected, with compilation errors on some systems, like Debian Jessie (EOL on June 30, 2020), I suggest to include this low impact PR in 0.18.

  2. fanquake added the label Build system on Mar 30, 2019
  3. MarcoFalke renamed this:
    Check QT library version
    build: Check QT library version
    on Mar 30, 2019
  4. MarcoFalke added the label Needs gitian build on Mar 30, 2019
  5. MarcoFalke added this to the milestone 0.18.1 on Mar 30, 2019
  6. MarcoFalke added the label Needs backport on Mar 30, 2019
  7. DrahtBot removed the label Needs gitian build on Mar 31, 2019
  8. fanquake requested review from theuni on Apr 16, 2019
  9. laanwj commented at 6:40 pm on May 16, 2019: member
    Concept ACK, can you take a look at the build system changes please @theuni
  10. in build-aux/m4/bitcoin_qt.m4:443 in 9d74d35dfc outdated 
    429+        PKG_CHECK_EXISTS(Qt5Core < 5.5.1,[bitcoin_cv_qt55=no],[bitcoin_cv_qt55=yes])
430+      )
431+    ])
432+    BITCOIN_QT_CHECK([
433+      AC_CACHE_CHECK(for > Qt 5.7,bitcoin_cv_qt58,
434+        PKG_CHECK_EXISTS(Qt5Core < 5.8.0,[bitcoin_cv_qt58=no],[bitcoin_cv_qt58=yes])
    laanwj commented at 2:56 pm on June 6, 2019:
    Message should mention > Qt 5.8 to be consistent
    lucayepa commented at 2:27 pm on July 19, 2019:

    I think it is consistent. “Check that QT > 5.7” means that it is 5.8 or more. This is the same as above: “check that QT > 5.4”, that means it is 5.5 or more. The same message is used some rows above (275):

    0AC_CACHE_CHECK(for > Qt 5.7, bitcoin_cv_qt58,[
1AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[
    laanwj commented at 12:26 pm on September 30, 2019:

    I think that’s confusing. > Qt 5.7 means newer than 5.7, so may also mean 5.7.1 or whatever, at least at a cursory glance.

    If you’re concerned about lawyering correctness of > versus >= feel free to make it >=5.8 instead.

    lucayepa commented at 9:44 pm on October 3, 2019:
    I think you are right. 5.7.1 is a good example. I’ll amend the commit.
  11. MarcoFalke deleted a comment on Jun 16, 2019
  12. MarcoFalke added the label Needs gitian build on Jun 16, 2019
  13. laanwj added the label Waiting for author on Jun 17, 2019
  14. DrahtBot removed the label Needs gitian build on Jun 18, 2019
  15. MarcoFalke deleted a comment on Jun 18, 2019
  16. MarcoFalke removed this from the milestone 0.18.1 on Jul 18, 2019
  17. laanwj added this to the milestone 0.18.2 on Jul 18, 2019
  18. MarcoFalke commented at 7:36 pm on July 18, 2019: member
    @lucayepa There has been a comment from @laanwj. Mind to address it?
  19. MarcoFalke deleted a comment on Jul 18, 2019
  20. fanquake removed the label Waiting for author on Sep 19, 2019
  21. fanquake requested review from dongcarl on Sep 19, 2019
  22. MarcoFalke added the label Waiting for author on Sep 30, 2019
  23. MarcoFalke removed the label Needs backport on Sep 30, 2019
  24. MarcoFalke removed this from the milestone 0.18.2 on Sep 30, 2019
  25. lucayepa force-pushed on Oct 3, 2019
  26. MarcoFalke removed the label Waiting for author on Oct 3, 2019
  27. MarcoFalke added the label Needs gitian build on Oct 3, 2019
  28. lucayepa commented at 10:50 pm on October 3, 2019: contributor

    I tested it on an affected machine.

    Are we in time to add back the 0.18.2 tag?

  29. laanwj added the label Needs backport (0.19) on Oct 4, 2019
  30. laanwj added the label Needs backport (0.18) on Oct 4, 2019
  31. laanwj added this to the milestone 0.18.2 on Oct 4, 2019
  32. DrahtBot commented at 2:39 pm on October 4, 2019: member

    Gitian builds for commit a689c119076c7b8dc5b4dea4539e4cbf5adfb72f (master):

    Gitian builds for commit 7afe2eaa8ed75e0aa3657653abcddfae2c65c651 (master and this pull):

  33. DrahtBot removed the label Needs gitian build on Oct 4, 2019
  34. lucayepa referenced this in commit 050b3e6152 on Oct 4, 2019
  35. lucayepa referenced this in commit f1b6f527bf on Oct 4, 2019
  36. laanwj commented at 10:17 am on October 5, 2019: member
    @dongcarl @theuni can one of you take a look here please
  37. in build-aux/m4/bitcoin_qt.m4:102 in 25e43fcd48 outdated 
     96@@ -99,6 +97,12 @@ AC_DEFUN([BITCOIN_QT_CONFIGURE],[
 97     BITCOIN_QT_CHECK([_BITCOIN_QT_FIND_LIBS_WITHOUT_PKGCONFIG])
 98   fi
 99 
100+  BITCOIN_QT_CHECK([
101+    if test "x$bitcoin_cv_qt551" != xyes; then
102+      BITCOIN_QT_FAIL([Qt version not supported])
    luke-jr commented at 2:13 pm on November 4, 2019:
    configure shouldn’t fail unless –with-gui is explicitly specified. Otherwise, it should just turn off GUI support…
    lucayepa commented at 8:28 pm on November 4, 2019:

    Yes, this is exactly the behavior. BITCOIN_QT_FAIL fails only if GUI has been requested. If not, it generates a warning. On a machine with Qt version 5.3.2 I get:

    0configure                 -> WARNING: Qt version not supported; bitcoin-qt frontend will not be built
1configure --with-gui      -> error: Qt version not supported
2configure --without-gui   -> no errors nor warnings
  38. luke-jr changes_requested
  39. hebasto commented at 9:52 pm on December 28, 2019: member
    Concept ACK.
  40. in build-aux/m4/bitcoin_qt.m4:259 in 25e43fcd48 outdated 
    254@@ -252,9 +255,9 @@ dnl ----
255 
256 dnl Internal. Check included version of Qt against minimum specified in doc/dependencies.md
257 dnl Requires: INCLUDES must be populated as necessary.
258-dnl Output: bitcoin_cv_qt5=yes|no
259-AC_DEFUN([_BITCOIN_QT_CHECK_QT5],[
260-  AC_CACHE_CHECK(for Qt 5, bitcoin_cv_qt5,[
261+dnl Output: bitcoin_cv_qt551=yes|no
262+AC_DEFUN([_BITCOIN_QT_CHECK_QT551],[
    hebasto commented at 9:57 pm on December 28, 2019: 
    0AC_DEFUN([_BITCOIN_QT_CHECK_MINIMUM_REQUIRED],[

    … to minimize future changes.

    lucayepa commented at 10:07 pm on January 24, 2020:
    Ack.
  41. in build-aux/m4/bitcoin_qt.m4:260 in 25e43fcd48 outdated 
    254@@ -252,9 +255,9 @@ dnl ----
255 
256 dnl Internal. Check included version of Qt against minimum specified in doc/dependencies.md
257 dnl Requires: INCLUDES must be populated as necessary.
258-dnl Output: bitcoin_cv_qt5=yes|no
259-AC_DEFUN([_BITCOIN_QT_CHECK_QT5],[
260-  AC_CACHE_CHECK(for Qt 5, bitcoin_cv_qt5,[
261+dnl Output: bitcoin_cv_qt551=yes|no
262+AC_DEFUN([_BITCOIN_QT_CHECK_QT551],[
263+  AC_CACHE_CHECK(for Qt >= 5.5.1, bitcoin_cv_qt551,[
    hebasto commented at 9:59 pm on December 28, 2019: 
    0  AC_CACHE_CHECK(for Qt >= 5.5.1, bitcoin_cv_qtminimumrequired,[

    … to minimize future changes.

    lucayepa commented at 10:07 pm on January 24, 2020:
    Ack
  42. in build-aux/m4/bitcoin_qt.m4:408 in 25e43fcd48 outdated 
    406@@ -404,8 +407,6 @@ AC_DEFUN([_BITCOIN_QT_FIND_STATIC_PLUGINS],[
407 dnl Internal. Find Qt libraries using pkg-config.
408 dnl Inputs: bitcoin_qt_want_version (from --with-gui=). The version to check
    hebasto commented at 10:05 pm on December 28, 2019:
    Is this comment still relevant?
    lucayepa commented at 10:14 pm on January 24, 2020:
    No. It is not. I remove it.
  43. hebasto changes_requested
  44. hebasto commented at 10:12 pm on December 28, 2019: member
    Please quote macro arguments with [ and ] consistently.
  45. DrahtBot commented at 1:28 am on December 29, 2019: member

    The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

    Conflicts

    Reviewers, this pull request conflicts with the following ones:

    • #18361 (build: Make the help string for –with-gui configure option unambiguous by hebasto)
    • #18298 (build: Fix Qt processing of configure script for depends with DEBUG=1 by hebasto)
    • #18297 (build: Use pkg-config in BITCOIN_QT_CONFIGURE for all hosts including Windows by hebasto)

    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.

  46. in build-aux/m4/bitcoin_qt.m4:432 in 25e43fcd48 outdated 
    426@@ -426,14 +427,25 @@ AC_DEFUN([_BITCOIN_QT_FIND_LIBS_WITH_PKGCONFIG],[
427         PKG_CHECK_MODULES([QT_DBUS], [${QT_LIB_PREFIX}DBus], [QT_DBUS_INCLUDES="$QT_DBUS_CFLAGS"; have_qt_dbus=yes], [have_qt_dbus=no])
428       fi
429     ])
430+    BITCOIN_QT_CHECK([
431+      AC_CACHE_CHECK(for Qt >= 5.5.1,bitcoin_cv_qt551,
432+        PKG_CHECK_EXISTS(Qt5Core < 5.5.1,[bitcoin_cv_qt551=no],[bitcoin_cv_qt551=yes])
    hebasto commented at 9:24 pm on December 29, 2019:

    https://cgit.freedesktop.org/pkg-config/tree/pkg.m4.in#n81

    0dnl PKG_CHECK_EXISTS(MODULES, [ACTION-IF-FOUND], [ACTION-IF-NOT-FOUND])`
1dnl -------------------------------------------------------------------
2dnl Since: 0.18
3dnl
4dnl Check to see whether a particular set of modules exists. Similar to
5dnl PKG_CHECK_MODULES(), but does not set variables or print errors

    Does this usage of PKG_CHECK_EXISTS macro is correct?

    hebasto commented at 1:19 pm on January 11, 2020:
    nm, a version specifier is ok.
  47. in build-aux/m4/bitcoin_qt.m4:443 in 25e43fcd48 outdated 
    432+        PKG_CHECK_EXISTS(Qt5Core < 5.5.1,[bitcoin_cv_qt551=no],[bitcoin_cv_qt551=yes])
433+      )
434+    ])
435+    BITCOIN_QT_CHECK([
436+      AC_CACHE_CHECK(for Qt >= 5.8.0,bitcoin_cv_qt58,
437+        PKG_CHECK_EXISTS(Qt5Core < 5.8.0,[bitcoin_cv_qt58=no],[bitcoin_cv_qt58=yes])
    hebasto commented at 9:25 pm on December 29, 2019:
    Does this usage of PKG_CHECK_EXISTS macro is correct? (the same as https://github.com/bitcoin/bitcoin/pull/15706/files#r361873134)
  48. jonasschnelli assigned theuni on Jan 5, 2020
  49. jonasschnelli assigned dongcarl on Jan 5, 2020
  50. jonasschnelli commented at 8:58 pm on January 5, 2020: contributor
    Concept ACK Assigned to @theuni / @dongcarl
  51. Check QT library version 
    Fixes issue #15688

Due to a bug, in systems using pkg-config, the version of the Qt library is
not checked at configure time. Without any check, when Qt version is not
supported, the build process stops with unexpleined errors.

This PR introduces the control of the version of the QT library, returning
a warning or an error at configure time, when the installed version is not
supported.
    35c46bcb68
  52. lucayepa force-pushed on Jan 24, 2020
  53. in build-aux/m4/bitcoin_qt.m4:438 in 35c46bcb68 
    432@@ -434,14 +433,25 @@ AC_DEFUN([_BITCOIN_QT_FIND_LIBS_WITH_PKGCONFIG],[
433         PKG_CHECK_MODULES([QT_DBUS], [${QT_LIB_PREFIX}DBus], [QT_DBUS_INCLUDES="$QT_DBUS_CFLAGS"; have_qt_dbus=yes], [have_qt_dbus=no])
434       fi
435     ])
436+    BITCOIN_QT_CHECK([
437+      AC_CACHE_CHECK(for Qt >= 5.5.1,bitcoin_cv_qt_minimumrequired,
438+        PKG_CHECK_EXISTS(Qt5Core < 5.5.1,[bitcoin_cv_qt_minimumrequired=no],[bitcoin_cv_qt_minimumrequired=yes])
    hebasto commented at 9:09 am on January 26, 2020:

    non-blocking nit, for consistency with “for Qt >= 5.5.1”:

    0        PKG_CHECK_EXISTS(Qt5Core >= 5.5.1,[bitcoin_cv_qt_minimumrequired=yes],[bitcoin_cv_qt_minimumrequired=no])
  54. in build-aux/m4/bitcoin_qt.m4:443 in 35c46bcb68 
    438+        PKG_CHECK_EXISTS(Qt5Core < 5.5.1,[bitcoin_cv_qt_minimumrequired=no],[bitcoin_cv_qt_minimumrequired=yes])
439+      )
440+    ])
441+    BITCOIN_QT_CHECK([
442+      AC_CACHE_CHECK(for Qt >= 5.8.0,bitcoin_cv_qt58,
443+        PKG_CHECK_EXISTS(Qt5Core < 5.8.0,[bitcoin_cv_qt58=no],[bitcoin_cv_qt58=yes])
    hebasto commented at 9:12 am on January 26, 2020:

    non-blocking nit, for consistency with “for Qt >= 5.8.0”:

    0        PKG_CHECK_EXISTS(Qt5Core >= 5.8.0,[bitcoin_cv_qt58=yes],[bitcoin_cv_qt58=no])
  55. in build-aux/m4/bitcoin_qt.m4:265 in 35c46bcb68 
    259@@ -257,9 +260,9 @@ dnl ----
260 
261 dnl Internal. Check included version of Qt against minimum specified in doc/dependencies.md
262 dnl Requires: INCLUDES must be populated as necessary.
263-dnl Output: bitcoin_cv_qt5=yes|no
264-AC_DEFUN([_BITCOIN_QT_CHECK_QT5],[
265-  AC_CACHE_CHECK(for Qt 5, bitcoin_cv_qt5,[
266+dnl Output: bitcoin_cv_qt_minimumrequired=yes|no
267+AC_DEFUN([_BITCOIN_QT_CHECK_MINIMUM_REQUIRED],[
268+  AC_CACHE_CHECK(for Qt >= 5.5.1, bitcoin_cv_qt_minimumrequired,[
    hebasto commented at 9:16 am on January 26, 2020:

    non-blocking nit: here and in other places mind quoting the first argument:

    0  AC_CACHE_CHECK([for Qt >= 5.5.1], bitcoin_cv_qt_minimumrequired,[

    ?

  56. in build-aux/m4/bitcoin_qt.m4:285 in 35c46bcb68 
    284+dnl Internal. Check if the included version of Qt is >= 5.8.
285 dnl Requires: INCLUDES must be populated as necessary.
286 dnl Output: bitcoin_cv_qt58=yes|no
287 AC_DEFUN([_BITCOIN_QT_CHECK_QT58],[
288-  AC_CACHE_CHECK(for > Qt 5.7, bitcoin_cv_qt58,[
289+  AC_CACHE_CHECK(for >= Qt 5.8, bitcoin_cv_qt58,[
    hebasto commented at 9:17 am on January 26, 2020: 
    0  AC_CACHE_CHECK([for >= Qt 5.8], bitcoin_cv_qt58,[
  57. in build-aux/m4/bitcoin_qt.m4:437 in 35c46bcb68 
    432@@ -434,14 +433,25 @@ AC_DEFUN([_BITCOIN_QT_FIND_LIBS_WITH_PKGCONFIG],[
433         PKG_CHECK_MODULES([QT_DBUS], [${QT_LIB_PREFIX}DBus], [QT_DBUS_INCLUDES="$QT_DBUS_CFLAGS"; have_qt_dbus=yes], [have_qt_dbus=no])
434       fi
435     ])
436+    BITCOIN_QT_CHECK([
437+      AC_CACHE_CHECK(for Qt >= 5.5.1,bitcoin_cv_qt_minimumrequired,
    hebasto commented at 9:18 am on January 26, 2020: 
    0      AC_CACHE_CHECK([for Qt >= 5.5.1],bitcoin_cv_qt_minimumrequired,
  58. in build-aux/m4/bitcoin_qt.m4:442 in 35c46bcb68 
    437+      AC_CACHE_CHECK(for Qt >= 5.5.1,bitcoin_cv_qt_minimumrequired,
438+        PKG_CHECK_EXISTS(Qt5Core < 5.5.1,[bitcoin_cv_qt_minimumrequired=no],[bitcoin_cv_qt_minimumrequired=yes])
439+      )
440+    ])
441+    BITCOIN_QT_CHECK([
442+      AC_CACHE_CHECK(for Qt >= 5.8.0,bitcoin_cv_qt58,
    hebasto commented at 9:20 am on January 26, 2020: 
    0      AC_CACHE_CHECK([for Qt >= 5.8.0],bitcoin_cv_qt58,
  59. hebasto approved
  60. hebasto commented at 9:26 am on January 26, 2020: member

    ACK 35c46bcb68344bb692dcc71a7c4dd6ac4442a920, tested on Linux Mint 19.3 with system Qt 5.9.5:

    0$ qmake -qt=qt5_system -v
1QMake version 3.1
2Using Qt version 5.9.5 in /usr/lib/x86_64-linux-gnu

    Tests with pkg-config usage

     0$ ./autogen.sh 
 1$ ./configure
 2...
 3checking for QT5... yes
 4checking for QT_TEST... yes
 5checking for QT_DBUS... yes
 6checking for Qt >= 5.5.1... yes
 7checking for Qt >= 5.8.0... yes
 8checking for static Qt... no
 9checking whether -fPIE can be used with this Qt config... no
10checking for moc-qt5... no
11checking for moc5... no
12checking for moc... /usr/lib/qt5/bin/moc
13checking for uic-qt5... no
14checking for uic5... no
15checking for uic... /usr/lib/qt5/bin/uic
16checking for rcc-qt5... no
17checking for rcc5... no
18checking for rcc... /usr/lib/qt5/bin/rcc
19checking for lrelease-qt5... no
20checking for lrelease5... no
21checking for lrelease... /usr/lib/qt5/bin/lrelease
22checking for lupdate-qt5... no
23checking for lupdate5... no
24checking for lupdate... /usr/lib/qt5/bin/lupdate
25checking whether to build Bitcoin Core GUI... yes (Qt5)
26...

    To test alternative behavior the following patch was applied:

     0$ git diff
 1diff --git a/build-aux/m4/bitcoin_qt.m4 b/build-aux/m4/bitcoin_qt.m4
 2index 019a3ebfb..be20602f0 100644
 3--- a/build-aux/m4/bitcoin_qt.m4
 4+++ b/build-aux/m4/bitcoin_qt.m4
 5@@ -262,7 +262,7 @@ dnl Internal. Check included version of Qt against minimum specified in doc/depe
 6 dnl Requires: INCLUDES must be populated as necessary.
 7 dnl Output: bitcoin_cv_qt_minimumrequired=yes|no
 8 AC_DEFUN([_BITCOIN_QT_CHECK_MINIMUM_REQUIRED],[
 9-  AC_CACHE_CHECK(for Qt >= 5.5.1, bitcoin_cv_qt_minimumrequired,[
10+  AC_CACHE_CHECK(for Qt >= 5.9.6, bitcoin_cv_qt_minimumrequired,[
11   AC_COMPILE_IFELSE([AC_LANG_PROGRAM([[
12       #include <QtCore/qconfig.h>
13       #ifndef QT_VERSION
14@@ -270,7 +270,7 @@ AC_DEFUN([_BITCOIN_QT_CHECK_MINIMUM_REQUIRED],[
15       #endif
16     ]],
17     [[
18-      #if QT_VERSION < 0x050501
19+      #if QT_VERSION < 0x050906
20       choke
21       #endif
22     ]])],
23@@ -434,8 +434,8 @@ AC_DEFUN([_BITCOIN_QT_FIND_LIBS_WITH_PKGCONFIG],[
24       fi
25     ])
26     BITCOIN_QT_CHECK([
27-      AC_CACHE_CHECK(for Qt >= 5.5.1,bitcoin_cv_qt_minimumrequired,
28-        PKG_CHECK_EXISTS(Qt5Core < 5.5.1,[bitcoin_cv_qt_minimumrequired=no],[bitcoin_cv_qt_minimumrequired=yes])
29+      AC_CACHE_CHECK(for Qt >= 5.9.6,bitcoin_cv_qt_minimumrequired,
30+        PKG_CHECK_EXISTS(Qt5Core < 5.9.6,[bitcoin_cv_qt_minimumrequired=no],[bitcoin_cv_qt_minimumrequired=yes])
31       )
32     ])
33     BITCOIN_QT_CHECK([

    0$ ./autogen.sh 
1$ ./configure
2checking for QT5... yes
3checking for QT_TEST... yes
4checking for QT_DBUS... yes
5checking for Qt >= 5.9.6... no
6checking for Qt >= 5.8.0... yes
7configure: WARNING: Qt version not supported; bitcoin-qt frontend will not be built
8checking whether to build Bitcoin Core GUI... no
9...

    0$ ./autogen.sh 
1$ ./configure --with-gui=qt5
2...
3checking for QT5... yes
4checking for QT_TEST... yes
5checking for QT_DBUS... yes
6checking for Qt >= 5.9.6... no
7checking for Qt >= 5.8.0... yes
8configure: error: Qt version not supported

    Tests without pkg-config usage

    For these tests the following patch was applied:

     0$ git diff configure.ac
 1diff --git a/configure.ac b/configure.ac
 2index 18f3104ac..d6c8745c7 100644
 3--- a/configure.ac
 4+++ b/configure.ac
 5@@ -479,7 +479,7 @@ AC_ARG_WITH([daemon],
 6   [build_bitcoind=$withval],
 7   [build_bitcoind=yes])
 8
 9-use_pkgconfig=yes
10+use_pkgconfig=no
11 case $host in
12   *mingw*)

    and --with-qt-incdir command-line option was passed to configure script.

    0$ ./autogen.sh
1$ ./configure --with-qt-incdir=/usr/include/x86_64-linux-gnu/qt5
2...
3checking for Qt >= 5.5.1... yes
4checking for >= Qt 5.8... yes
5...
6checking whether to build Bitcoin Core GUI... yes (Qt5)
7...

    0$ ./autogen.sh
1$ ./configure --with-qt-incdir=/usr/include/x86_64-linux-gnu/qt5
2...
3checking for Qt >= 5.9.6... no
4checking for >= Qt 5.8... yes
5...
6configure: WARNING: Qt version not supported; bitcoin-qt frontend will not be built
7checking whether to build Bitcoin Core GUI... no
8...
  61. laanwj removed this from the milestone 0.18.2 on Jan 27, 2020
  62. hebasto commented at 10:30 am on March 17, 2020: member
    It seems we should consider an alternative solution based on #18297.
  63. fanquake commented at 7:26 am on May 15, 2020: member
    I agree that something like #18297 is the way forward here, so I’m going to close. This also would not be getting backported to 0.18 or 0.19 at this point. Thanks for the contribution & sorry it didn’t make it into the repo.
  64. fanquake closed this on May 15, 2020
  65. fanquake removed the label Needs backport (0.18) on May 15, 2020
  66. fanquake removed the label Needs backport (0.19) on May 15, 2020
  67. fanquake referenced this in commit 38c13a4a60 on Aug 24, 2020
  68. sidhujag referenced this in commit dafb919f5c on Aug 24, 2020
  69. xdustinface referenced this in commit 3e8687603d on Feb 17, 2021
  70. gades referenced this in commit 0e12d032a0 on Jun 27, 2021
  71. DrahtBot locked this on Feb 15, 2022


lucayepa laanwj MarcoFalke DrahtBot luke-jr hebasto jonasschnelli fanquake


theuni dongcarl

Labels
Build system

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bitcoin. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2025-06-30 18:13 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me