Skip to content

[30.x] Partial backport #34650#34754

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

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

Conversation

@hebasto
Copy link
Member

@hebasto hebasto commented Mar 6, 2026

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
Copy link
Contributor

DrahtBot commented Mar 6, 2026

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
Copy link
Member Author

hebasto commented Mar 6, 2026

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
Copy link
Member Author

hebasto commented Mar 7, 2026

Undrafted.

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

fanquake commented Mar 9, 2026

ACK fbfc778

@willcl-ark
Copy link
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
Copy link
Member

fanquake commented Mar 9, 2026

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