Skip to content

[30.x] Partial backport #34650#34754

Merged
fanquake merged 4 commits into
bitcoin:30.xfrom
hebasto:260306-qt6-gcc16
Mar 9, 2026
Merged

[30.x] Partial backport #34650#34754
fanquake merged 4 commits into
bitcoin:30.xfrom
hebasto:260306-qt6-gcc16

Conversation

@hebasto

@hebasto hebasto commented Mar 6, 2026

Copy link
Copy Markdown
Member

This PR backports commits from #34650 not related to Qt update. They fix issues when building with GCC 16 and improves Qt detecting.

The `find_package(Qt .. MODULE REQUIRED COMPONENTS ...)` call must treat
any missing component as a fatal error.

Github-Pull: bitcoin#34650
Rebased-From: c555845
@DrahtBot

DrahtBot commented Mar 6, 2026

Copy link
Copy Markdown
Contributor

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

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK fanquake, willcl-ark

If your review is incorrectly listed, please copy-paste <!--meta-tag:bot-skip--> into the comment that the bot should ignore.

@fanquake fanquake added this to the 30.3 milestone Mar 6, 2026
@hebasto

hebasto commented Mar 6, 2026

Copy link
Copy Markdown
Member Author

Looking into an error when cross-compiling for ARM32...

@hebasto hebasto force-pushed the 260306-qt6-gcc16 branch from ef15c61 to fbfc778 Compare March 6, 2026 18:10
@hebasto

hebasto commented Mar 7, 2026

Copy link
Copy Markdown
Member Author

Undrafted.

@hebasto hebasto marked this pull request as ready for review March 7, 2026 20:17
@fanquake

fanquake commented Mar 9, 2026

Copy link
Copy Markdown
Member

ACK fbfc778

@willcl-ark

Copy link
Copy Markdown
Member

utACK fbfc778

This PR backports commits from #34650 not related to Qt update. They fix issues when building with GCC 16 and improves Qt detecting.

Makes sense to drop this QT bump but take the fixes on a stable branch.

Rest of the backports look correct, mainly differing on patch metadata.

It is missing a release note, is that intentional as it's "partial"?

@fanquake

fanquake commented Mar 9, 2026

Copy link
Copy Markdown
Member

It is missing a release note, is that intentional as it's "partial"?

I think we can add a partial release note in #34689.

@fanquake fanquake merged commit 62730a8 into bitcoin:30.x Mar 9, 2026
20 checks passed
@hebasto hebasto deleted the 260306-qt6-gcc16 branch March 9, 2026 15:53
morozow pushed a commit to morozow/bitcoin_rd that referenced this pull request May 8, 2026
25dc6d0 qt: Add patch to fix SFINAE warnings in QAnyStringView with gcc16 (Hennadii Stepanov)
db5fdc1 qt: add patches to fix SFINAE errors/warnings with gcc16 (Cory Fields)
63820a4 qt: add patch to fix build with gcc16 (Cory Fields)
8048c70 cmake: Fix `FindQt` module (Hennadii Stepanov)

Pull request description:

  This PR backports commits from bitcoin#34650 not related to Qt update. They fix issues when building with GCC 16 and improves Qt detecting.

ACKs for top commit:
  fanquake:
    ACK 25dc6d0
  willcl-ark:
    utACK 25dc6d0

Tree-SHA512: dd2d1b23aaad64a8eb6a279dbb7def1b29149cfdebd88e0667ebbccfd30ad9762f90dbbc6968eae148e724f0257646939360765c2fae1b5d513a11cf0c2902d8
@fanquake fanquake mentioned this pull request Jun 3, 2026
sedited added a commit that referenced this pull request Jun 23, 2026
d52747d doc: update manual pages for v30.3rc1 (fanquake)
a1406d6 build: bump version to v30.3rc1 (fanquake)
1bdd2b1 doc: update release notes for v30.3rc1 (fanquake)
e7c6d39 fuzz: Remove unused g_setup pointers (MarcoFalke)
b7c7d1e coins: compact chainstate in background (Lőrinc)
992b1dd validation: randomly compact chainstate (Lőrinc)
55ead70 coins: test chainstate flush baseline (Lőrinc)
d09bd0f lint: disable leveldb subtree check (fanquake)

Pull request description:

  Backports:
  * #34918
  * #35465

  Final changes for v`30.3rc1`.

  Backports since `30.2`:
  * #34283
  * #34459
  * #34689
  * #34754
  * #34805
  * #34856
  * #35211
  * #35232

ACKs for top commit:
  sedited:
    ACK d52747d
  marcofleon:
    ACK d52747d

Tree-SHA512: 75fb64f744a0689a9bc4f5a25128bf2aaad7459d7a430f552200b995498428d89de28820f477780f95e7e8485eea9f657ad5fdeae253940df4f29bf51346b4ff
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.

5 participants