build: Bump libevent minimum version up to 2.1.8 #24681

pull hebasto wants to merge 3 commits into bitcoin:master from hebasto:220323-libevent changing 7 files +12 −51
  1. hebasto commented at 8:00 am on March 26, 2022: member

    Required to support new functionality in bitcoin/bitcoin#19420.

    libevent availability: https://repology.org/project/libevent/versions

  2. hebasto commented at 8:01 am on March 26, 2022: member
    cc @promag
  3. hebasto force-pushed on Mar 26, 2022
  4. hebasto renamed this:
    build: Bump libevent version up to 2.1.8
    build: Bump libevent minimum version up to 2.1.8
    on Mar 26, 2022
  5. build: Bump libevent minimum version up to 2.1.8
    Required to support new functionality.
    aaf72d62c1
  6. hebasto force-pushed on Mar 26, 2022
  7. DrahtBot added the label Build system on Mar 26, 2022
  8. promag commented at 10:25 am on March 26, 2022: member
    ACK aaf72d62c18f9cb325c150cf0cc21abb201607c8 🙏
  9. prusnak commented at 11:27 am on March 26, 2022: contributor

    For reference:

    • 2.0.21 was released in 2012
    • 2.1.8 was released in 2017
  10. DrahtBot commented at 4:23 pm on March 27, 2022: 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:

    • #24671 (build: remove unneeded configure $*val setting by fanquake)
    • #24291 (build: Remove negated –enable-fuzz checks from build system by MarcoFalke)
    • #23969 (build: remove use of TARGET_OS and BUILD_OS by fanquake)

    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.

  11. MarcoFalke added the label DrahtBot Guix build requested on Mar 29, 2022
  12. fanquake commented at 9:59 am on March 30, 2022: member
    I think requiring 2.1.8 from 24.x onwards is ok, if there is something we want to use, but shouldn’t this PR have more changes? The version number for the 2.1.8 release is 0x02010800, so all our #if LIBEVENT_VERSION_NUMBER >= 0x02010300 conditions should go, as well as a workaround in the fuzz tests. See branch here: https://github.com/fanquake/bitcoin/tree/account_for_new_libevent_minimum.
  13. hebasto commented at 11:29 am on March 30, 2022: member

    @fanquake

    I think requiring 2.1.8 from 24.x onwards is ok, if there is something we want to use, but shouldn’t this PR have more changes? The version number for the 2.1.8 release is 0x02010800, so all our #if LIBEVENT_VERSION_NUMBER >= 0x02010300 conditions should go, as well as a workaround in the fuzz tests. See branch here: https://github.com/fanquake/bitcoin/tree/account_for_new_libevent_minimum.

    Thanks! Added.

  14. refactor: account for requiring libevent 2.1.8+ 0598f36852
  15. in src/bitcoin-cli.cpp:747 in a362c43875 outdated
    742@@ -747,9 +743,8 @@ static UniValue CallRPC(BaseRequestHandler* rh, const std::string& strMethod, co
    743     raii_evhttp_request req = obtain_evhttp_request(http_request_done, (void*)&response);
    744     if (req == nullptr)
    745         throw std::runtime_error("create http request failed");
    746-#if LIBEVENT_VERSION_NUMBER >= 0x02010300
    747+
    


    MarcoFalke commented at 11:54 am on March 30, 2022:
    0}
    

    nit: Maybe add {} while touching the line?


    hebasto commented at 12:03 pm on March 30, 2022:
    Thanks! Updated.
  16. hebasto force-pushed on Mar 30, 2022
  17. hebasto commented at 12:02 pm on March 30, 2022: member

    Updated a362c43875b0974c23238e55bc3f21c224fe84c0 -> 0598f36852199d0cee8fe9e676a2e0bec3ebf624 (pr24681.03 -> pr24681.04, diff):

  18. hebasto commented at 1:56 pm on March 30, 2022: member

    Guix builds on x86_64:

     0$ find guix-build-$(git rev-parse --short=12 HEAD)/output/ -type f -print0 | env LC_ALL=C sort -z | xargs -r0 sha256sum
     1b7738a0c94aaaa58a2bc8846b60c37c565184451c92dc6b856c0dc9d5e0d4197  guix-build-0598f3685219/output/aarch64-linux-gnu/SHA256SUMS.part
     24808ec36d368cf65846e4de6e7e206f5d262e55522511bf80c6267f4011a8f4c  guix-build-0598f3685219/output/aarch64-linux-gnu/bitcoin-0598f3685219-aarch64-linux-gnu-debug.tar.gz
     36897a48e2289581f065673730a09fdd6697a00542f340cd965a39755c07b7549  guix-build-0598f3685219/output/aarch64-linux-gnu/bitcoin-0598f3685219-aarch64-linux-gnu.tar.gz
     4df617c76e0bc96e3e543ccc374bcdf7e080037fd48960e9569c5190a33d14499  guix-build-0598f3685219/output/arm-linux-gnueabihf/SHA256SUMS.part
     54292972310cb72b4ed7a22e631577826ff46bf23afbebbed17e1ed8d23a7d20e  guix-build-0598f3685219/output/arm-linux-gnueabihf/bitcoin-0598f3685219-arm-linux-gnueabihf-debug.tar.gz
     6f346d3acd2bdcb76e6e1f59a98d71ddc5827b6efa78527b23955f1e75ac3df5d  guix-build-0598f3685219/output/arm-linux-gnueabihf/bitcoin-0598f3685219-arm-linux-gnueabihf.tar.gz
     73690a84b8272491a0b455695c4e6bd7223d6f8f499db910c8476f35ed07724c2  guix-build-0598f3685219/output/arm64-apple-darwin/SHA256SUMS.part
     8363d343c74e3a60d18c8fc22fc7b52a6af9157c9f6f106654be3fa3036365930  guix-build-0598f3685219/output/arm64-apple-darwin/bitcoin-0598f3685219-arm64-apple-darwin-unsigned.dmg
     96491aec3025c45ee95a917062e1e044e128754ca420ed5dfb7de37558a7b7499  guix-build-0598f3685219/output/arm64-apple-darwin/bitcoin-0598f3685219-arm64-apple-darwin-unsigned.tar.gz
    10eca770ebc37ca081b2a88247738a001df16a34e61f2e997c268b306625bd38b9  guix-build-0598f3685219/output/arm64-apple-darwin/bitcoin-0598f3685219-arm64-apple-darwin.tar.gz
    119ebb415c90efc517a603e68131ff06f9c7442e95e0d2e3b08f65523792922f42  guix-build-0598f3685219/output/dist-archive/bitcoin-0598f3685219.tar.gz
    128d62b2932975f864348f113c482ce19ebf601c03c12d1952a3b53ff163f43203  guix-build-0598f3685219/output/powerpc64-linux-gnu/SHA256SUMS.part
    13a897b71ab60bf0150620ea90da601ebd9414305e7789ebfe68020e58e3642166  guix-build-0598f3685219/output/powerpc64-linux-gnu/bitcoin-0598f3685219-powerpc64-linux-gnu-debug.tar.gz
    14d9f9506286b5ef3dceee31d9742c24b89f1140e92e1a8df98da381b64c2d767b  guix-build-0598f3685219/output/powerpc64-linux-gnu/bitcoin-0598f3685219-powerpc64-linux-gnu.tar.gz
    15924fa61ff89895d0711222056907617b23a761e0ae56063423dacd0a3de20cce  guix-build-0598f3685219/output/powerpc64le-linux-gnu/SHA256SUMS.part
    1676a0719f342f44741f99435b3ff5576af74b5d55c39a1af64dbd536f56ab52f6  guix-build-0598f3685219/output/powerpc64le-linux-gnu/bitcoin-0598f3685219-powerpc64le-linux-gnu-debug.tar.gz
    178423d01ac8efa523047b0b895f55e0efda6289666ad7526b92ade2c192605949  guix-build-0598f3685219/output/powerpc64le-linux-gnu/bitcoin-0598f3685219-powerpc64le-linux-gnu.tar.gz
    182c9848faecff7e9f9c711f9ac32fa6deceee401a883b72a11d117b4f8d1d34f7  guix-build-0598f3685219/output/riscv64-linux-gnu/SHA256SUMS.part
    191060ae38588ca98782a6fe7e0763e565fd1a115c7dc6f42b97be95f4039e48a3  guix-build-0598f3685219/output/riscv64-linux-gnu/bitcoin-0598f3685219-riscv64-linux-gnu-debug.tar.gz
    20f7401fdc281e85b9ee2ae9f578d03aba0ee641aebdf18fcebf44cac88ffb0a52  guix-build-0598f3685219/output/riscv64-linux-gnu/bitcoin-0598f3685219-riscv64-linux-gnu.tar.gz
    21d46b89599cea6b2959de8ea2e9dc5a37ad0ac973b837c95acba879c79820a9d6  guix-build-0598f3685219/output/x86_64-apple-darwin/SHA256SUMS.part
    22564a392ca2f66f354d2d9c934b0539e9b787eb1f8c8679b9ab9a9281532ff1e0  guix-build-0598f3685219/output/x86_64-apple-darwin/bitcoin-0598f3685219-x86_64-apple-darwin-unsigned.dmg
    2394bdb7b18764d0a9b6898e977e6a93b2fef9e435e08c3186b4dedb21780b460a  guix-build-0598f3685219/output/x86_64-apple-darwin/bitcoin-0598f3685219-x86_64-apple-darwin-unsigned.tar.gz
    2483aad0ef1701537bf40f75d5440f1757b3d2234850f095c24f902bacedc6422e  guix-build-0598f3685219/output/x86_64-apple-darwin/bitcoin-0598f3685219-x86_64-apple-darwin.tar.gz
    25757a62da0e2f658eecc820c11007e73323ce3c7c1bc1128b42204a763a0028bd  guix-build-0598f3685219/output/x86_64-linux-gnu/SHA256SUMS.part
    268cd9dedf07a2a5f0f0e8f28f3786e417bc8d1c260ef02ce9734fc3c27a1d34fb  guix-build-0598f3685219/output/x86_64-linux-gnu/bitcoin-0598f3685219-x86_64-linux-gnu-debug.tar.gz
    27dbdc614531ad45ba76b87fe9ce22c0365241f29e5b24d4fcab5e80194199a7df  guix-build-0598f3685219/output/x86_64-linux-gnu/bitcoin-0598f3685219-x86_64-linux-gnu.tar.gz
    28251bd427c27a94e34159eb133425e1193ddfa07bdfa372947d2d428b7331cdad  guix-build-0598f3685219/output/x86_64-w64-mingw32/SHA256SUMS.part
    296f8cd99f21f7d6f327532d8a6ece209e3182691b48c2fa777b0f7ef24051906d  guix-build-0598f3685219/output/x86_64-w64-mingw32/bitcoin-0598f3685219-win64-debug.zip
    304d1a88a15cea2a3e227f6a6b6add3ce1c49a4bbfcf20c2e2df4e94efa3b43084  guix-build-0598f3685219/output/x86_64-w64-mingw32/bitcoin-0598f3685219-win64-setup-unsigned.exe
    316d53f8afb56ee66ca97ac8137ab571ced604cb765bf19170278fd1f680187b46  guix-build-0598f3685219/output/x86_64-w64-mingw32/bitcoin-0598f3685219-win64-unsigned.tar.gz
    3293f44aa2fa255f577bc13f2cc008f73d15ba2cee0e9797d3cb8084008107ef76  guix-build-0598f3685219/output/x86_64-w64-mingw32/bitcoin-0598f3685219-win64.zip
    
  19. jonatack commented at 2:33 pm on March 30, 2022: member

    Concept ACK per #19420 (comment).

    utACK 0598f36852199d0cee8fe9e676a2e0bec3ebf624

    Light-GitHub-and-grep-locally-review while waiting on a build of another pull; it looks like all the instances of LIBEVENT_VERSION_NUMBER, 0x02010100 and = 2.0.21 have been handled

  20. DrahtBot removed the label DrahtBot Guix build requested on Mar 31, 2022
  21. MarcoFalke added the label DrahtBot Guix build requested on Mar 31, 2022
  22. MarcoFalke deleted a comment on Mar 31, 2022
  23. laanwj commented at 10:00 am on April 1, 2022: member
    A reason to not bump the minimum libevent from 2.0.x was that distributions didn’t ship libevent 2.1.x versions (due to some unclarity about beta versus stable). Did this get resolved, do distros now reliably ship 2.1.8? or above?
  24. prusnak commented at 10:05 am on April 1, 2022: contributor

    Did this get resolved, do distros now reliably ship 2.1.8? or above?

    Yes, see the OP: https://repology.org/project/libevent/versions

    Also 2.1.8 was released in 2017.

  25. laanwj commented at 10:06 am on April 1, 2022: member

    Concept ACK then.

    Also 2.1.8 was released in 2017.

    That unfortunately meant very little while distros didn’t want any 2.1 version, and 2.0 was effectively dead. But good to know that changed.

  26. prusnak approved
  27. prusnak commented at 10:06 am on April 1, 2022: contributor
    Concept ACK
  28. in src/test/fuzz/http_request.cpp:22 in 0598f36852 outdated
    18@@ -19,23 +19,8 @@
    19 #include <string>
    20 #include <vector>
    21 
    22-// workaround for libevent versions before 2.1.1,
    


    laanwj commented at 10:11 am on April 1, 2022:
    Nice to get rid of cruft like this.
  29. laanwj added this to the milestone 24.0 on Apr 1, 2022
  30. fanquake commented at 1:07 pm on April 2, 2022: member

    Guix Build (on x86_64):

     0b7738a0c94aaaa58a2bc8846b60c37c565184451c92dc6b856c0dc9d5e0d4197  guix-build-0598f3685219/output/aarch64-linux-gnu/SHA256SUMS.part
     14808ec36d368cf65846e4de6e7e206f5d262e55522511bf80c6267f4011a8f4c  guix-build-0598f3685219/output/aarch64-linux-gnu/bitcoin-0598f3685219-aarch64-linux-gnu-debug.tar.gz
     26897a48e2289581f065673730a09fdd6697a00542f340cd965a39755c07b7549  guix-build-0598f3685219/output/aarch64-linux-gnu/bitcoin-0598f3685219-aarch64-linux-gnu.tar.gz
     3df617c76e0bc96e3e543ccc374bcdf7e080037fd48960e9569c5190a33d14499  guix-build-0598f3685219/output/arm-linux-gnueabihf/SHA256SUMS.part
     44292972310cb72b4ed7a22e631577826ff46bf23afbebbed17e1ed8d23a7d20e  guix-build-0598f3685219/output/arm-linux-gnueabihf/bitcoin-0598f3685219-arm-linux-gnueabihf-debug.tar.gz
     5f346d3acd2bdcb76e6e1f59a98d71ddc5827b6efa78527b23955f1e75ac3df5d  guix-build-0598f3685219/output/arm-linux-gnueabihf/bitcoin-0598f3685219-arm-linux-gnueabihf.tar.gz
     63690a84b8272491a0b455695c4e6bd7223d6f8f499db910c8476f35ed07724c2  guix-build-0598f3685219/output/arm64-apple-darwin/SHA256SUMS.part
     7363d343c74e3a60d18c8fc22fc7b52a6af9157c9f6f106654be3fa3036365930  guix-build-0598f3685219/output/arm64-apple-darwin/bitcoin-0598f3685219-arm64-apple-darwin-unsigned.dmg
     86491aec3025c45ee95a917062e1e044e128754ca420ed5dfb7de37558a7b7499  guix-build-0598f3685219/output/arm64-apple-darwin/bitcoin-0598f3685219-arm64-apple-darwin-unsigned.tar.gz
     9eca770ebc37ca081b2a88247738a001df16a34e61f2e997c268b306625bd38b9  guix-build-0598f3685219/output/arm64-apple-darwin/bitcoin-0598f3685219-arm64-apple-darwin.tar.gz
    109ebb415c90efc517a603e68131ff06f9c7442e95e0d2e3b08f65523792922f42  guix-build-0598f3685219/output/dist-archive/bitcoin-0598f3685219.tar.gz
    118d62b2932975f864348f113c482ce19ebf601c03c12d1952a3b53ff163f43203  guix-build-0598f3685219/output/powerpc64-linux-gnu/SHA256SUMS.part
    12a897b71ab60bf0150620ea90da601ebd9414305e7789ebfe68020e58e3642166  guix-build-0598f3685219/output/powerpc64-linux-gnu/bitcoin-0598f3685219-powerpc64-linux-gnu-debug.tar.gz
    13d9f9506286b5ef3dceee31d9742c24b89f1140e92e1a8df98da381b64c2d767b  guix-build-0598f3685219/output/powerpc64-linux-gnu/bitcoin-0598f3685219-powerpc64-linux-gnu.tar.gz
    14924fa61ff89895d0711222056907617b23a761e0ae56063423dacd0a3de20cce  guix-build-0598f3685219/output/powerpc64le-linux-gnu/SHA256SUMS.part
    1576a0719f342f44741f99435b3ff5576af74b5d55c39a1af64dbd536f56ab52f6  guix-build-0598f3685219/output/powerpc64le-linux-gnu/bitcoin-0598f3685219-powerpc64le-linux-gnu-debug.tar.gz
    168423d01ac8efa523047b0b895f55e0efda6289666ad7526b92ade2c192605949  guix-build-0598f3685219/output/powerpc64le-linux-gnu/bitcoin-0598f3685219-powerpc64le-linux-gnu.tar.gz
    172c9848faecff7e9f9c711f9ac32fa6deceee401a883b72a11d117b4f8d1d34f7  guix-build-0598f3685219/output/riscv64-linux-gnu/SHA256SUMS.part
    181060ae38588ca98782a6fe7e0763e565fd1a115c7dc6f42b97be95f4039e48a3  guix-build-0598f3685219/output/riscv64-linux-gnu/bitcoin-0598f3685219-riscv64-linux-gnu-debug.tar.gz
    19f7401fdc281e85b9ee2ae9f578d03aba0ee641aebdf18fcebf44cac88ffb0a52  guix-build-0598f3685219/output/riscv64-linux-gnu/bitcoin-0598f3685219-riscv64-linux-gnu.tar.gz
    20d46b89599cea6b2959de8ea2e9dc5a37ad0ac973b837c95acba879c79820a9d6  guix-build-0598f3685219/output/x86_64-apple-darwin/SHA256SUMS.part
    21564a392ca2f66f354d2d9c934b0539e9b787eb1f8c8679b9ab9a9281532ff1e0  guix-build-0598f3685219/output/x86_64-apple-darwin/bitcoin-0598f3685219-x86_64-apple-darwin-unsigned.dmg
    2294bdb7b18764d0a9b6898e977e6a93b2fef9e435e08c3186b4dedb21780b460a  guix-build-0598f3685219/output/x86_64-apple-darwin/bitcoin-0598f3685219-x86_64-apple-darwin-unsigned.tar.gz
    2383aad0ef1701537bf40f75d5440f1757b3d2234850f095c24f902bacedc6422e  guix-build-0598f3685219/output/x86_64-apple-darwin/bitcoin-0598f3685219-x86_64-apple-darwin.tar.gz
    24757a62da0e2f658eecc820c11007e73323ce3c7c1bc1128b42204a763a0028bd  guix-build-0598f3685219/output/x86_64-linux-gnu/SHA256SUMS.part
    258cd9dedf07a2a5f0f0e8f28f3786e417bc8d1c260ef02ce9734fc3c27a1d34fb  guix-build-0598f3685219/output/x86_64-linux-gnu/bitcoin-0598f3685219-x86_64-linux-gnu-debug.tar.gz
    26dbdc614531ad45ba76b87fe9ce22c0365241f29e5b24d4fcab5e80194199a7df  guix-build-0598f3685219/output/x86_64-linux-gnu/bitcoin-0598f3685219-x86_64-linux-gnu.tar.gz
    27251bd427c27a94e34159eb133425e1193ddfa07bdfa372947d2d428b7331cdad  guix-build-0598f3685219/output/x86_64-w64-mingw32/SHA256SUMS.part
    286f8cd99f21f7d6f327532d8a6ece209e3182691b48c2fa777b0f7ef24051906d  guix-build-0598f3685219/output/x86_64-w64-mingw32/bitcoin-0598f3685219-win64-debug.zip
    294d1a88a15cea2a3e227f6a6b6add3ce1c49a4bbfcf20c2e2df4e94efa3b43084  guix-build-0598f3685219/output/x86_64-w64-mingw32/bitcoin-0598f3685219-win64-setup-unsigned.exe
    306d53f8afb56ee66ca97ac8137ab571ced604cb765bf19170278fd1f680187b46  guix-build-0598f3685219/output/x86_64-w64-mingw32/bitcoin-0598f3685219-win64-unsigned.tar.gz
    3193f44aa2fa255f577bc13f2cc008f73d15ba2cee0e9797d3cb8084008107ef76  guix-build-0598f3685219/output/x86_64-w64-mingw32/bitcoin-0598f3685219-win64.zip
    
  31. refactor: Remove outdated libevent logging code
    The removed code was intended to catch issues with event_enable_debug_logging which was not available prior to libevent 2.1.1. This is not necessary since the minimum libevent version was bumped to 2.1.8.
    e40779a4fe
  32. fjahr commented at 5:33 pm on April 2, 2022: member

    Concept ACK and light code review ACK 0598f36852199d0cee8fe9e676a2e0bec3ebf624

    I think I have found another piece of code that can be refactored after the bump: https://github.com/fjahr/bitcoin/commit/e40779a4fee03c6c455149bd8e9d1a7ccd991450

    If this is as straight forward as it seems to me, feel free to pull the commit in here. Otherwise I can open a follow-up PR.

  33. prusnak commented at 5:36 pm on April 2, 2022: contributor

    I think I have found another piece of code that can be refactored after the bump: fjahr@e40779a

    +1 for including this change into the PR

  34. DrahtBot commented at 4:18 pm on April 3, 2022: member

    Guix builds

    File commit 1a54c060b3b30220ec40a4b3cc4a0eb43586e982(master) commit 7c036607a98c4988e08a51ee174fb1b20b38b3ed(master and this pull)
    SHA256SUMS.part bffaae95c9b700ac... d9686855f4e8c023...
    *-aarch64-linux-gnu-debug.tar.gz 82b9ebf8b1408349... e8909787628cca76...
    *-aarch64-linux-gnu.tar.gz 41df6a28f176a570... d0f79bd403794cf1...
    *-arm-linux-gnueabihf-debug.tar.gz 562d60a38050a57e... 4217b661cece6439...
    *-arm-linux-gnueabihf.tar.gz a3ce174d1b85975d... 5b8865325357e8e5...
    *-arm64-apple-darwin-unsigned.dmg 2d61bc717a85d115... 79f62082fe0751cc...
    *-arm64-apple-darwin-unsigned.tar.gz 481bd89de9d37477... 11154ab13abc89ff...
    *-arm64-apple-darwin.tar.gz 00e5560fc575e52d... 4d8fb1dc11b74395...
    *-powerpc64-linux-gnu-debug.tar.gz ad48bc3250bec98e... e4518764d3f27987...
    *-powerpc64-linux-gnu.tar.gz b5b37090b6544f94... 6fc5814db796549a...
    *-powerpc64le-linux-gnu-debug.tar.gz cad227c0f3102883... 88550285ff883885...
    *-powerpc64le-linux-gnu.tar.gz 91bb4973e374e227... 403ca01306a20000...
    *-riscv64-linux-gnu-debug.tar.gz d78c521f02573f42... f74839aec727a41b...
    *-riscv64-linux-gnu.tar.gz ace83bdc3a4ce594... 768c76c8cd268f1f...
    *-win64-debug.zip f7013d3f1fdce85b... 04c8eb01603bc313...
    *-win64-setup-unsigned.exe f5d35f796509960a... 94c7227e624bef47...
    *-win64-unsigned.tar.gz 7414d20470a751f1... 9ec0f39f29637e56...
    *-win64.zip 2ff99ea1e6ee3e7d... 4cecc3bf7ce864eb...
    *-x86_64-apple-darwin-unsigned.dmg 637f820c47cb0676... 97cdde0e41bc4f04...
    *-x86_64-apple-darwin-unsigned.tar.gz 6acf8c650c5c4222... 040d7c1205f0a5e3...
    *-x86_64-apple-darwin.tar.gz 8d8d208ca2b0f4df... 378d49134294335e...
    *-x86_64-linux-gnu-debug.tar.gz 43f4223dc95d752a... e83305773d3d99e5...
    *-x86_64-linux-gnu.tar.gz 96ccc43790171174... 9526d66f6caefdd7...
    *.tar.gz 6c3a0ad29dac027c... 7b0f230de398ac73...
    guix_build.log 92c6d7928257143e... c669b7449adc68bc...
    guix_build.log.diff 95cb267c63663a86...
  35. DrahtBot removed the label DrahtBot Guix build requested on Apr 3, 2022
  36. jarolrod commented at 6:36 am on April 6, 2022: member

    GUIX hashes on x86, mine match fanquake’s and hebasto’s latest round;

     0$ find guix-build-$(git rev-parse --short=12 HEAD)/output/ -type f -print0 | env LC_ALL=C sort -z | xargs -r0 sha256sum
     1b7738a0c94aaaa58a2bc8846b60c37c565184451c92dc6b856c0dc9d5e0d4197  guix-build-0598f3685219/output/aarch64-linux-gnu/SHA256SUMS.part
     24808ec36d368cf65846e4de6e7e206f5d262e55522511bf80c6267f4011a8f4c  guix-build-0598f3685219/output/aarch64-linux-gnu/bitcoin-0598f3685219-aarch64-linux-gnu-debug.tar.gz
     36897a48e2289581f065673730a09fdd6697a00542f340cd965a39755c07b7549  guix-build-0598f3685219/output/aarch64-linux-gnu/bitcoin-0598f3685219-aarch64-linux-gnu.tar.gz
     4df617c76e0bc96e3e543ccc374bcdf7e080037fd48960e9569c5190a33d14499  guix-build-0598f3685219/output/arm-linux-gnueabihf/SHA256SUMS.part
     54292972310cb72b4ed7a22e631577826ff46bf23afbebbed17e1ed8d23a7d20e  guix-build-0598f3685219/output/arm-linux-gnueabihf/bitcoin-0598f3685219-arm-linux-gnueabihf-debug.tar.gz
     6f346d3acd2bdcb76e6e1f59a98d71ddc5827b6efa78527b23955f1e75ac3df5d  guix-build-0598f3685219/output/arm-linux-gnueabihf/bitcoin-0598f3685219-arm-linux-gnueabihf.tar.gz
     73690a84b8272491a0b455695c4e6bd7223d6f8f499db910c8476f35ed07724c2  guix-build-0598f3685219/output/arm64-apple-darwin/SHA256SUMS.part
     8363d343c74e3a60d18c8fc22fc7b52a6af9157c9f6f106654be3fa3036365930  guix-build-0598f3685219/output/arm64-apple-darwin/bitcoin-0598f3685219-arm64-apple-darwin-unsigned.dmg
     96491aec3025c45ee95a917062e1e044e128754ca420ed5dfb7de37558a7b7499  guix-build-0598f3685219/output/arm64-apple-darwin/bitcoin-0598f3685219-arm64-apple-darwin-unsigned.tar.gz
    10eca770ebc37ca081b2a88247738a001df16a34e61f2e997c268b306625bd38b9  guix-build-0598f3685219/output/arm64-apple-darwin/bitcoin-0598f3685219-arm64-apple-darwin.tar.gz
    119ebb415c90efc517a603e68131ff06f9c7442e95e0d2e3b08f65523792922f42  guix-build-0598f3685219/output/dist-archive/bitcoin-0598f3685219.tar.gz
    128d62b2932975f864348f113c482ce19ebf601c03c12d1952a3b53ff163f43203  guix-build-0598f3685219/output/powerpc64-linux-gnu/SHA256SUMS.part
    13a897b71ab60bf0150620ea90da601ebd9414305e7789ebfe68020e58e3642166  guix-build-0598f3685219/output/powerpc64-linux-gnu/bitcoin-0598f3685219-powerpc64-linux-gnu-debug.tar.gz
    14d9f9506286b5ef3dceee31d9742c24b89f1140e92e1a8df98da381b64c2d767b  guix-build-0598f3685219/output/powerpc64-linux-gnu/bitcoin-0598f3685219-powerpc64-linux-gnu.tar.gz
    15924fa61ff89895d0711222056907617b23a761e0ae56063423dacd0a3de20cce  guix-build-0598f3685219/output/powerpc64le-linux-gnu/SHA256SUMS.part
    1676a0719f342f44741f99435b3ff5576af74b5d55c39a1af64dbd536f56ab52f6  guix-build-0598f3685219/output/powerpc64le-linux-gnu/bitcoin-0598f3685219-powerpc64le-linux-gnu-debug.tar.gz
    178423d01ac8efa523047b0b895f55e0efda6289666ad7526b92ade2c192605949  guix-build-0598f3685219/output/powerpc64le-linux-gnu/bitcoin-0598f3685219-powerpc64le-linux-gnu.tar.gz
    182c9848faecff7e9f9c711f9ac32fa6deceee401a883b72a11d117b4f8d1d34f7  guix-build-0598f3685219/output/riscv64-linux-gnu/SHA256SUMS.part
    191060ae38588ca98782a6fe7e0763e565fd1a115c7dc6f42b97be95f4039e48a3  guix-build-0598f3685219/output/riscv64-linux-gnu/bitcoin-0598f3685219-riscv64-linux-gnu-debug.tar.gz
    20f7401fdc281e85b9ee2ae9f578d03aba0ee641aebdf18fcebf44cac88ffb0a52  guix-build-0598f3685219/output/riscv64-linux-gnu/bitcoin-0598f3685219-riscv64-linux-gnu.tar.gz
    21d46b89599cea6b2959de8ea2e9dc5a37ad0ac973b837c95acba879c79820a9d6  guix-build-0598f3685219/output/x86_64-apple-darwin/SHA256SUMS.part
    22564a392ca2f66f354d2d9c934b0539e9b787eb1f8c8679b9ab9a9281532ff1e0  guix-build-0598f3685219/output/x86_64-apple-darwin/bitcoin-0598f3685219-x86_64-apple-darwin-unsigned.dmg
    2394bdb7b18764d0a9b6898e977e6a93b2fef9e435e08c3186b4dedb21780b460a  guix-build-0598f3685219/output/x86_64-apple-darwin/bitcoin-0598f3685219-x86_64-apple-darwin-unsigned.tar.gz
    2483aad0ef1701537bf40f75d5440f1757b3d2234850f095c24f902bacedc6422e  guix-build-0598f3685219/output/x86_64-apple-darwin/bitcoin-0598f3685219-x86_64-apple-darwin.tar.gz
    25757a62da0e2f658eecc820c11007e73323ce3c7c1bc1128b42204a763a0028bd  guix-build-0598f3685219/output/x86_64-linux-gnu/SHA256SUMS.part
    268cd9dedf07a2a5f0f0e8f28f3786e417bc8d1c260ef02ce9734fc3c27a1d34fb  guix-build-0598f3685219/output/x86_64-linux-gnu/bitcoin-0598f3685219-x86_64-linux-gnu-debug.tar.gz
    27dbdc614531ad45ba76b87fe9ce22c0365241f29e5b24d4fcab5e80194199a7df  guix-build-0598f3685219/output/x86_64-linux-gnu/bitcoin-0598f3685219-x86_64-linux-gnu.tar.gz
    28251bd427c27a94e34159eb133425e1193ddfa07bdfa372947d2d428b7331cdad  guix-build-0598f3685219/output/x86_64-w64-mingw32/SHA256SUMS.part
    296f8cd99f21f7d6f327532d8a6ece209e3182691b48c2fa777b0f7ef24051906d  guix-build-0598f3685219/output/x86_64-w64-mingw32/bitcoin-0598f3685219-win64-debug.zip
    304d1a88a15cea2a3e227f6a6b6add3ce1c49a4bbfcf20c2e2df4e94efa3b43084  guix-build-0598f3685219/output/x86_64-w64-mingw32/bitcoin-0598f3685219-win64-setup-unsigned.exe
    316d53f8afb56ee66ca97ac8137ab571ced604cb765bf19170278fd1f680187b46  guix-build-0598f3685219/output/x86_64-w64-mingw32/bitcoin-0598f3685219-win64-unsigned.tar.gz
    3293f44aa2fa255f577bc13f2cc008f73d15ba2cee0e9797d3cb8084008107ef76  guix-build-0598f3685219/output/x86_64-w64-mingw32/bitcoin-0598f3685219-win64.zip
    
  37. hebasto commented at 7:22 am on April 6, 2022: member

    @fjahr

    Concept ACK and light code review ACK 0598f36

    I think I have found another piece of code that can be refactored after the bump: fjahr@e40779a

    If this is as straight forward as it seems to me, feel free to pull the commit in here. Otherwise I can open a follow-up PR.

    Thanks! Added.

  38. laanwj commented at 10:47 am on April 6, 2022: member
    Code review ACK e40779a4fee03c6c455149bd8e9d1a7ccd991450
  39. fanquake commented at 12:14 pm on April 6, 2022: member

    Guix Build (on x86_64):

     061afdfbd14469eda67b2d915c1d9d387aa7cfa67c852f8b447f2a4c2ecefa64c  guix-build-e40779a4fee0/output/aarch64-linux-gnu/SHA256SUMS.part
     143976291608acfee150772329ef583dcca9bfc3f2d89b8b1f8ff770663472c35  guix-build-e40779a4fee0/output/aarch64-linux-gnu/bitcoin-e40779a4fee0-aarch64-linux-gnu-debug.tar.gz
     2cde2ab4b3655bdc35a3828547c202f356319d107270d10b28c37e2772d290c86  guix-build-e40779a4fee0/output/aarch64-linux-gnu/bitcoin-e40779a4fee0-aarch64-linux-gnu.tar.gz
     30eb72862a7f680c4274b9ff28dcd119206ff4a6c342d932cb606d3d9abb4da9c  guix-build-e40779a4fee0/output/arm-linux-gnueabihf/SHA256SUMS.part
     4fe122ba9e66d64a750f334f422bb74eea05dedc095cfb51558a623f60fb1d1c6  guix-build-e40779a4fee0/output/arm-linux-gnueabihf/bitcoin-e40779a4fee0-arm-linux-gnueabihf-debug.tar.gz
     5b413a3a3fa9e6e511332fa1ad25d53be4aa7a050edc485ecb055c975cbf5f60d  guix-build-e40779a4fee0/output/arm-linux-gnueabihf/bitcoin-e40779a4fee0-arm-linux-gnueabihf.tar.gz
     6decbd666758482433a0ded0c0e57552a47c33086096bb13c7877b000a83d07fd  guix-build-e40779a4fee0/output/arm64-apple-darwin/SHA256SUMS.part
     7bc041375b82629b4310238648080dfdcd6826b1631ddc6e7d4ca3243fabf42a4  guix-build-e40779a4fee0/output/arm64-apple-darwin/bitcoin-e40779a4fee0-arm64-apple-darwin-unsigned.dmg
     8ee5d1de4083488c00817f6f4b4a90b8ccc74b3caf29af0a3db49e04c00cf13d2  guix-build-e40779a4fee0/output/arm64-apple-darwin/bitcoin-e40779a4fee0-arm64-apple-darwin-unsigned.tar.gz
     9fcef23e20cc25dd99e9634875305a9a857a853d0a3bc3a88c51f5abb38241019  guix-build-e40779a4fee0/output/arm64-apple-darwin/bitcoin-e40779a4fee0-arm64-apple-darwin.tar.gz
    1022fad677c2e5f5610200f6fcfffe9bf61b1ca839799a34758832fc864d79459c  guix-build-e40779a4fee0/output/dist-archive/bitcoin-e40779a4fee0.tar.gz
    11f571e2c6b902c053289f77b06526f6ff84954e7054cf882695dbd9201c526142  guix-build-e40779a4fee0/output/powerpc64-linux-gnu/SHA256SUMS.part
    124df2cb2fa948627e78588a2e7ed9f260c04f56cdebcbd880c0f70e014c2cdb58  guix-build-e40779a4fee0/output/powerpc64-linux-gnu/bitcoin-e40779a4fee0-powerpc64-linux-gnu-debug.tar.gz
    13827e8f11b56ede4d236ad17d7031538cd80caa9223276058cbc6614061edcd00  guix-build-e40779a4fee0/output/powerpc64-linux-gnu/bitcoin-e40779a4fee0-powerpc64-linux-gnu.tar.gz
    14b1fbf0768494bd4167a5bd43ed8c6ca08a089ba8a1b29d164bc517e2c662ce6b  guix-build-e40779a4fee0/output/powerpc64le-linux-gnu/SHA256SUMS.part
    15b5d7a26d860749f300f36a22471c4226ab9ec140a3f192e456855b6d78e34b4b  guix-build-e40779a4fee0/output/powerpc64le-linux-gnu/bitcoin-e40779a4fee0-powerpc64le-linux-gnu-debug.tar.gz
    1630b1c76daea40d40d487f4bb328d6191bc8c63b24822bde55160e6480b3a20d8  guix-build-e40779a4fee0/output/powerpc64le-linux-gnu/bitcoin-e40779a4fee0-powerpc64le-linux-gnu.tar.gz
    1750aafa855382d39703d9c67b17d3e7e6c2efff80d56638d4fd9e3cafce98cb67  guix-build-e40779a4fee0/output/riscv64-linux-gnu/SHA256SUMS.part
    18e71f5df0132edfe1b01b1a58040ba724976c292c362faba594476d6418545d8c  guix-build-e40779a4fee0/output/riscv64-linux-gnu/bitcoin-e40779a4fee0-riscv64-linux-gnu-debug.tar.gz
    1955d86637c60efd705d19006c7811dd26a11cae499da722e914ab61b992274a90  guix-build-e40779a4fee0/output/riscv64-linux-gnu/bitcoin-e40779a4fee0-riscv64-linux-gnu.tar.gz
    2087ec8f81933aae0c79162ab4084c9212caa6a93851fda413834edd41fec4a8bd  guix-build-e40779a4fee0/output/x86_64-apple-darwin/SHA256SUMS.part
    210f858d52b37e6a5bda6b8bbd31c34a97e131696ae0803a183a760b1a72575ea5  guix-build-e40779a4fee0/output/x86_64-apple-darwin/bitcoin-e40779a4fee0-x86_64-apple-darwin-unsigned.dmg
    2225261f6401940a301ddb9c423bdf2c6c0721a86ae10a9e923264da7411ebec6e  guix-build-e40779a4fee0/output/x86_64-apple-darwin/bitcoin-e40779a4fee0-x86_64-apple-darwin-unsigned.tar.gz
    233bf028ca001a013d21216eaa773d68cb709fab697b1ae2525589d110ce1dc672  guix-build-e40779a4fee0/output/x86_64-apple-darwin/bitcoin-e40779a4fee0-x86_64-apple-darwin.tar.gz
    24e4eabbabf4c276684123b6240b92f8389906827ed1a5231153481b8b1ce6f3ea  guix-build-e40779a4fee0/output/x86_64-linux-gnu/SHA256SUMS.part
    2574f8cee7c92b3aa5b7ec9c4a7cc44eb140aa11ec269fa7c1cb0c905feaf1f47c  guix-build-e40779a4fee0/output/x86_64-linux-gnu/bitcoin-e40779a4fee0-x86_64-linux-gnu-debug.tar.gz
    26a16923f7262c1a2588d9fdac4a3aae08eb77f349a6e035ccbf76b4529c5b6f34  guix-build-e40779a4fee0/output/x86_64-linux-gnu/bitcoin-e40779a4fee0-x86_64-linux-gnu.tar.gz
    27b2b148cddf4d40611e488630c7999c0a8f1996bb3e7e3788546b0598a5f5e6d1  guix-build-e40779a4fee0/output/x86_64-w64-mingw32/SHA256SUMS.part
    282014b814af6d0b2c1382d2fb6c935d5b75a9b026c3e70679f867661461b1526c  guix-build-e40779a4fee0/output/x86_64-w64-mingw32/bitcoin-e40779a4fee0-win64-debug.zip
    2966c8606e8232aeb94f1f1c1bed00e8b22181128f3b5dbe4c332f6c6518751c29  guix-build-e40779a4fee0/output/x86_64-w64-mingw32/bitcoin-e40779a4fee0-win64-setup-unsigned.exe
    30741050ab2a4851c1510e23f5ad8f709f2318b55f2b6aa6dfac3da5d96c2ef36b  guix-build-e40779a4fee0/output/x86_64-w64-mingw32/bitcoin-e40779a4fee0-win64-unsigned.tar.gz
    314d1d951c73fabde7cd5443641e658c9478fa6e5050401058504e972abd6d856f  guix-build-e40779a4fee0/output/x86_64-w64-mingw32/bitcoin-e40779a4fee0-win64.zip
    
  40. fanquake approved
  41. fanquake commented at 12:15 pm on April 6, 2022: member
    ACK e40779a4fee03c6c455149bd8e9d1a7ccd991450
  42. fanquake merged this on Apr 6, 2022
  43. fanquake closed this on Apr 6, 2022

  44. hebasto deleted the branch on Apr 6, 2022
  45. sidhujag referenced this in commit 243bfa996c on Apr 6, 2022
  46. DrahtBot locked this on Apr 6, 2023

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: 2024-11-17 21:12 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me