depends: capnp 1.2.0 #32760

pull fanquake wants to merge 1 commits into bitcoin:master from fanquake:capnp_1_2_0 changing 5 files +2 −227

fanquake commented at 10:03 AM on June 16, 2025: member

See https://github.com/capnproto/capnproto/compare/release-1.1.0...release-1.2.0. We can drop all the patches we are currently applying.
depends: capnp 1.2.0 c7eaac326a
DrahtBot commented at 10:03 AM on June 16, 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/32760.



Reviews

See the guideline for information on the review process.

Type Reviewers

ACK theStack, ryanofsky, Sjors

Concept ACK hebasto

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 Build system on Jun 16, 2025
hebasto commented at 10:17 AM on June 16, 2025: member

Concept ACK.
hebasto commented at 10:36 AM on June 16, 2025: member

OpenBSD build looks fine.
theStack approved
theStack commented at 12:36 PM on June 16, 2025: contributor

ACK c7eaac326ac20a745d1bc6dbc6bc48c1b5eb46f8

Verified that the removed patches are contained in the 1.2.0 release and did a MULTIPROCESS=1 depends build locally on OpenBSD 7.7 (amd64).
DrahtBot requested review from hebasto on Jun 16, 2025
ryanofsky commented at 1:14 PM on June 16, 2025: contributor

Started reviewing this and ran into something strange. For me the download URL https://capnproto.org/capnproto-c++-1.2.0.tar.gz returns a 404, but adding an extra slash works https://capnproto.org//capnproto-c++-1.2.0.tar.gz. It seems like depends build does add the extra slash so does not have a problem (testing with make MULTIPROCESS=1 V=1 capnp_fetched). Same problem does not affect https://capnproto.org/capnproto-c++-1.1.0.tar.gz
ryanofsky approved
ryanofsky commented at 1:35 PM on June 16, 2025: contributor

Code review ACK c7eaac326ac20a745d1bc6dbc6bc48c1b5eb46f8. Just checked hashes, compared tarball to git and diffed 1.1.0 and 1.2.0 tarballs which showed only minor and expected changes.
Sjors commented at 2:46 PM on June 16, 2025: member

ACK c7eaac326ac20a745d1bc6dbc6bc48c1b5eb46f8

Tested on top of #31802 (comment) including with OpenBSD. Also tested a depends build on macOS 13.7.6 (M4) with https://github.com/Sjors/bitcoin/pull/48 by mining a (SRI signet) block.

Dropping our patches is nice. But otherwise I didn't review the upstream changes.
ryanofsky assigned ryanofsky on Jun 16, 2025
ryanofsky merged this on Jun 16, 2025
ryanofsky closed this on Jun 16, 2025
fanquake deleted the branch on Jun 16, 2025
fanquake commented at 9:44 AM on June 19, 2025: member

Backported to 29.x in #32589.
fanquake referenced this in commit ef6111bd18 on Jun 19, 2025
fanquake referenced this in commit 983b268768 on Jun 25, 2025
stringintech referenced this in commit f19e7819e9 on Jul 19, 2025
alexanderwiederin referenced this in commit 9997c3d993 on Jul 25, 2025
alexanderwiederin referenced this in commit 8d6ebb9c4e on Jul 28, 2025
alexanderwiederin referenced this in commit 533a4585b5 on Jul 28, 2025
yuvicc referenced this in commit 22f55cf11d on Aug 26, 2025
bug-castercv502 referenced this in commit d4bd8a1259 on Sep 28, 2025
stickies-v referenced this in commit a19c56cd7c on Nov 4, 2025
kwvg referenced this in commit 7bf7523aef on Jan 23, 2026
PastaPastaPasta referenced this in commit 1ea35a436d on Jan 24, 2026
morozow referenced this in commit 9f3b599d68 on May 8, 2026
morozow referenced this in commit c81a1e776a on May 8, 2026
Kino1994 referenced this in commit 3c0a6f5fa8 on Jun 28, 2026
BigcoinBGC referenced this in commit 703d1f8b54 on Jun 30, 2026
bitcoin locked this on Jul 30, 2026