Skip to content

Conversation

@fanquake
Copy link
Member

@fanquake fanquake commented Jun 4, 2024

Backports:

I don't think either of these changes warrants an rc2 cycle.

@fanquake fanquake added this to the 27.1 milestone Jun 4, 2024
@DrahtBot
Copy link
Contributor

DrahtBot commented Jun 4, 2024

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

Code Coverage

For detailed information about the code coverage, see the test coverage report.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK stickies-v, willcl-ark

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

@hebasto
Copy link
Member

hebasto commented Jun 4, 2024

Backports:

I don't think either of these changes warrants an rc2 cycle.

Agree.

@fanquake fanquake changed the title [27.x] rc2 or final [27.1] Finalize Jun 10, 2024
@fanquake fanquake marked this pull request as ready for review June 10, 2024 12:06
Copy link
Contributor

@stickies-v stickies-v left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK d756a38

I checked that:

  • backports are clean and look correct
  • manpages are identical to mine
  • release notes are correct

Copy link
Member

@willcl-ark willcl-ark left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ACK d756a38

Version updates and manpages LGTM.

Also ran a scripted backport check to verify no unwarranted changes:

output
₿ bitcoin-diff-backports --base-branch="upstream/27.x"
1c1
< commit ffbc173ca1ed6b93de8bf3f88b8aed0743f4916c
---
> commit f2e05cd2a9bdc42fa45d284a5162f1f670d07ab6
7a8,10
>
>     Github-Pull: #30217
>     Rebased-From: ffbc173ca1ed6b93de8bf3f88b8aed0743f4916c
10c13
< index 7f0389b30d..938e9971ba 100644
---
> index ab43764b38..5e61036227 100644
18,20c21,23
<  $(package)_file_name=boost_$(subst .,_,$($(package)_version)).tar.gz
<  $(package)_sha256_hash=205666dea9f6a7cfed87c7a6dfbeb52a2c1b9de55712c9c1a87735d7181452b6
<
---
>  $(package)_file_name=boost_$(subst .,_,$($(package)_version)).tar.bz2
>  $(package)_sha256_hash=71feeed900fbccca04a3b4f2f84a7c217186f28a940ed8b7ed4725986baf99fa
>  $(package)_patches=process_macos_sdk.patch
1c1
< commit 3299abce948f205bb1354993614b669189f9b89f
---
> commit ba359205421e2f765391a8330e8a61055acd87fc
5a6,8
>
>     Github-Pull: #30216
>     Rebased-From: 3299abce948f205bb1354993614b669189f9b89f
8c11
< index 62a82189c1..0cb3b185ad 100644
---
> index 9f9bdbbd0c..a8a52e3ca5 100644
11c14
< @@ -277,7 +277,7 @@ FUZZ_SUITE_LD_COMMON += $(LIBBITCOIN_ZMQ) $(ZMQ_LIBS)
---
> @@ -274,7 +274,7 @@ FUZZ_SUITE_LD_COMMON += $(LIBBITCOIN_ZMQ) $(ZMQ_LIBS)

@fanquake fanquake merged commit 1088a98 into bitcoin:27.x Jun 11, 2024
@fanquake fanquake deleted the 27_1_rc2_or_rel branch June 11, 2024 13:22
@bitcoin bitcoin locked and limited conversation to collaborators Jun 11, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants