Skip to content

Conversation

fanquake
Copy link
Member

@fanquake fanquake commented Jan 13, 2025

kehiy and others added 3 commits January 10, 2025 17:10
This change resolves an issue where a missing space caused the value of
the `build_AR` variable to be concatenated with the "NM=" string. This
resulted in subsequent calls to `${AR}` and `${NM}` failing.

Github-Pull: bitcoin#31627
Rebased-From: 8a46286
@fanquake fanquake added this to the 28.2 milestone Jan 13, 2025
@DrahtBot
Copy link
Contributor

DrahtBot commented Jan 13, 2025

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/31648.

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.

@luke-jr
Copy link
Member

luke-jr commented Jan 14, 2025

#31623 is a clean merge to both branches, so no need for a cherry-pick here

@fanquake fanquake marked this pull request as ready for review March 5, 2025 11:18
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 77c13c7

All commits look good to me.

I guess the commit title "[WIP] Update release notes" gets updated later during finalisation?

@fanquake
Copy link
Member Author

fanquake commented Mar 5, 2025

@willcl-ark I've just changed it here.

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 2c372fc

All backports are clean, except;

  • 4e7dd0a backported from b537a2c (due to switching to CMake), looks okay to me

Backports seem sensible.

nit:

  • #31611 missing from PR description
  • dd32974 doesn't have the Github-Pull and Rebased-From metadata
  • commit authors 0xB10C and kehiy missing from release notes

@DrahtBot DrahtBot requested a review from willcl-ark March 5, 2025 13:45
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 2c372fc

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 e57359c

@DrahtBot DrahtBot requested a review from willcl-ark March 5, 2025 14:25
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 e57359c

Now with nits addressed.

@glozow glozow merged commit dbc450c into bitcoin:28.x Mar 5, 2025
15 of 16 checks passed
@fanquake fanquake deleted the 28_x_backports branch March 5, 2025 17:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants