Backports:
[28.x] Backports #32299
pull fanquake wants to merge 6 commits into bitcoin:28.x from fanquake:28_x_backports changing 11 files +56 −25-
fanquake commented at 2:27 pm on April 17, 2025: member
-
test: Handle empty string returned by CLI as None in RPC tests 9b15b20cb1
-
refactor: Remove spurious virtual from final ~CZMQNotificationInterface
Github-Pull: #32187 Rebased-From: fa69c42fdf0aeec0546e951bc6132ab630edb9d4
-
build: use make < 3.82 syntax for define directive
From the GNU make 3.82 release announcement: * The 'define' make directive now allows a variable assignment operator after the variable name, to allow for simple, conditional, or appending multi-line variable assignment. macOS ships with 3.81. This caused the multiprocess config options to be ignored. Fixes #32068 Co-authored-by: Ryan Ofsky <ryan@ofsky.org> Github-Pull: #32070 Rebased-From: 9157d9e449870851ef455e077249ac46fc2df24c
-
fanquake added this to the milestone 28.2 on Apr 17, 2025
-
DrahtBot commented at 2:27 pm on April 17, 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/32299.
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.
-
DrahtBot added the label Backport on Apr 17, 2025
-
DrahtBot added the label CI failed on Apr 17, 2025
-
test: Add imports for util bpf_cflags
This is required for the next commit. Github-Pull: #32336 Rebased-From: fa0c1baaf89805182de56d6a2e8eed2265a94b40
-
fanquake force-pushed on Apr 24, 2025
-
scripted-diff: Use bpf_cflags
-BEGIN VERIFY SCRIPT- ren() { sed --regexp-extended -i "s/$1/$2/g" $( git grep --extended-regexp -l "$1" ) ; } ren 'cflags=\["-Wno-error=implicit-function-declaration"\]' 'cflags=bpf_cflags()' -END VERIFY SCRIPT- Github-Pull: #32336 Rebased-From: facb9b327b9da39ce1e09ed56199be9efb19b5b8
-
fanquake force-pushed on Apr 24, 2025
-
fanquake marked this as ready for review on May 12, 2025
-
fanquake requested review from willcl-ark on May 12, 2025
-
willcl-ark approved
-
willcl-ark commented at 11:52 am on May 13, 2025: member
crACK a325ca34653872534529e7934d07fe4e2ca29cae
Backports all look clean to me.
Would only check that #32336 was left out of the release notes intentionally (it’s not really “notable”)?
-
doc: update release notes for 28.x 812e637521
-
fanquake force-pushed on May 13, 2025
-
fanquake commented at 11:57 am on May 13, 2025: member
was left out of the release notes
Fixed that up.
-
willcl-ark approved
-
willcl-ark commented at 12:00 pm on May 13, 2025: member
ACK 812e6375213e883a0c22aa926ebcde5da4d23a3e
Only changes to release note since previous ACK.
-
fanquake merged this on May 13, 2025
-
fanquake closed this on May 13, 2025
-
fanquake deleted the branch on May 13, 2025
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-07 21:13 UTC
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-07 21:13 UTC
This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me
More mirrored repositories can be found on mirror.b10c.me