[28.x] More backports #32735

pull fanquake wants to merge 3 commits into bitcoin:28.x from fanquake:28_x_win_sign_cmake_4 changing 5 files +39 −9
  1. fanquake commented at 9:45 am on June 12, 2025: member
  2. fanquake added this to the milestone 28.2 on Jun 12, 2025
  3. DrahtBot commented at 9:45 am on June 12, 2025: contributor

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

    Code Coverage & Benchmarks

    For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32735.

    Reviews

    See the guideline for information on the review process.

    Type Reviewers
    ACK willcl-ark

    If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

    LLM Linter (✨ experimental)

    Possible typos and grammar issues:

    • C_VISBILITY_PRESET -> C_VISIBILITY_PRESET [correct spelling of “visibility” in the CMake property name]
    • accomodate -> accommodate [correct spelling of “accommodate”]

    drahtbot_id_4_m

  4. DrahtBot added the label Backport on Jun 12, 2025
  5. DrahtBot added the label CI failed on Jun 12, 2025
  6. DrahtBot commented at 11:42 am on June 12, 2025: contributor

    🚧 At least one of the CI tasks failed. Task Win64, unit tests, no gui tests, no functional tests: https://github.com/bitcoin/bitcoin/runs/43959393827 LLM reason (✨ experimental): The failure is caused by errors in the test suite due to issues with Wine compatibility and missing system DLLs.

    Try to run the tests locally, according to the documentation. However, a CI failure may still happen due to a number of reasons, for example:

    • Possibly due to a silent merge conflict (the changes in this pull request being incompatible with the current code in the target branch). If so, make sure to rebase on the latest commit of the target branch.

    • A sanitizer issue, which can only be found by compiling with the sanitizer and running the affected test.

    • An intermittent issue.

    Leave a comment here, if you need help tracking down a confusing failure.

  7. contrib: Sign all Windows binaries too
    Github-Pull: #31407
    Rebased-From: 3656b828dc2204418974e94928cc8d915b10ed95
    9082498589
  8. build: patch cmake min version on freetype
    Patch cmake_minimum_required version for freetype, using the version from
    https://gitlab.freedesktop.org/freetype/freetype/-/merge_requests/352/diffs
    
    This fixes a failure when building with CMake 4, where compatibility
    with CMake versions < 3.5 has been removed (see https://cmake.org/cmake/help/latest/release/4.0.html#deprecated-and-removed-features)
    
    Github-Pull: #32693
    Rebased-From: d7c37906e7b1dc1513439056d0cdb2bd341d00f1
    a6aca67214
  9. doc: update 28.x release notes 2437d93989
  10. fanquake force-pushed on Jun 13, 2025
  11. fanquake marked this as ready for review on Jun 16, 2025
  12. fanquake requested review from willcl-ark on Jun 16, 2025
  13. fanquake requested review from stickies-v on Jun 17, 2025
  14. willcl-ark approved
  15. willcl-ark commented at 12:01 pm on June 17, 2025: member

    ACK 2437d9398926b82f2015d78752a72d8df82772f2

    Commit 9082498589f09b4cd81ec017d67b684fa6f0efd9 “contrib: Sign all Windows binaries too” was missed from a previous backport, and 32693 is backported correctly.

  16. fanquake merged this on Jun 17, 2025
  17. fanquake closed this on Jun 17, 2025

  18. fanquake deleted the branch on Jun 17, 2025
  19. stickies-v commented at 12:59 pm on June 17, 2025: contributor

    Post-merge ACK 2437d9398926b82f2015d78752a72d8df82772f2

    All backport commits are clean, and make sense.

  20. josibake commented at 1:52 pm on June 17, 2025: member

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-07-07 12:13 UTC

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