Skip to content

[29.x] Finalise 29.2rc2#33534

Merged
glozow merged 4 commits into
bitcoin:29.xfrom
fanquake:final_29_2_rc2
Oct 3, 2025
Merged

[29.x] Finalise 29.2rc2#33534
glozow merged 4 commits into
bitcoin:29.xfrom
fanquake:final_29_2_rc2

Conversation

@fanquake

@fanquake fanquake commented Oct 3, 2025

Copy link
Copy Markdown
Member

@fanquake fanquake added this to the 29.2 milestone Oct 3, 2025
@DrahtBot

DrahtBot commented Oct 3, 2025

Copy link
Copy Markdown
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/33534.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK glozow, davidgumberg, darosior

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

amisha-chhajed and others added 3 commits October 3, 2025 16:26
QT translations are optional, but the script would error when
'translations_dir' falls back to its default value NULL.

This PR fixes it by moving the set-up of QT translations under
the check for 'translations_dir' presence.

Github-Pull: bitcoin#33482
Rebased-From: 7b5261f
@davidgumberg

davidgumberg commented Oct 3, 2025

Copy link
Copy Markdown
Contributor

ACK 513cef7

I compared this to other PR's (#32766) of the "finalize format", and this has the two finalizing chagnes:

  • this bumps the build version in the 29.x branch
  • updates the versions in the manpages.

I compared the backport commit 6b3c1db to 7b5261f from #33482.

@glozow glozow left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

lgtm 513cef7. Got the same manpages, no changes to examples/bitcoin.conf or bips.md.

I think the release notes need to be updated

Comment thread contrib/macdeploy/macdeployqtplus

@glozow glozow left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

ACK d82fc69

@DrahtBot DrahtBot requested a review from davidgumberg October 3, 2025 17:10
@davidgumberg

Copy link
Copy Markdown
Contributor

reACK d82fc69

@darosior darosior left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

utACK d82fc69. Changes look good to me, but i have not been through the process of regenerating the doc myself.

@DrahtBot DrahtBot requested a review from darosior October 3, 2025 17:15
@glozow glozow merged commit 398c176 into bitcoin:29.x Oct 3, 2025
19 checks passed
@fanquake fanquake deleted the final_29_2_rc2 branch October 3, 2025 18:12
@glozow glozow mentioned this pull request Oct 9, 2025
14 tasks
morozow pushed a commit to morozow/bitcoin_rd that referenced this pull request May 8, 2026
a735fe5 doc: update release notes for 29.2rc2 (fanquake)
69cd3ef doc: update manual pages for v29.2rc2 (fanquake)
f7f8689 build: bump version to v29.2rc2 (fanquake)
ed62c32 contrib: fix using macdploy script without translations. (amisha)

Pull request description:

  It's been 2 weeks since rc1: https://github.com/bitcoin/bitcoin/releases/tag/v29.2rc1.
  We've backported more changes:
  * bitcoin#33403
  * bitcoin#33474
  * bitcoin#33482

  Lets do `rc2`.

ACKs for top commit:
  davidgumberg:
    reACK a735fe5
  glozow:
    ACK a735fe5
  darosior:
    utACK a735fe5. Changes look good to me, but i have not been through the process of regenerating the doc myself.

Tree-SHA512: c829efe89f86c9c76767ffe60a3779ece902ee9e3c8f6b4203562aaf257019484bfa49916ddfabdcabbd1478368d9b80a3f0a15057778aa1984852ea245283a6
ivanlele added a commit to ivanlele/elements that referenced this pull request Jul 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants