-
Notifications
You must be signed in to change notification settings - Fork 1.2k
depends: Qt 5.15.18 #6949
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
depends: Qt 5.15.18 #6949
Conversation
✅ No Merge Conflicts DetectedThis PR currently has no conflicts with other open PRs. |
|
Guix Automation has began to build this PR tagged as v23.0.0-devpr6949.daa18446. A new comment will be made when the image is pushed. |
Checksums for daa1844 |
|
Guix Automation has completed; a release should be present here: https://github.com/dashpay/dash-dev-branches/releases/tag/v23.0.0-devpr6949.daa18446. The image should be on dockerhub soon. |
Checksums for 5046964 |
|
Guix Automation has began to build this PR tagged as v23.0.0-devpr6949.5046964d. A new comment will be made when the image is pushed. |
|
Guix Automation has completed; a release should be present here: https://github.com/dashpay/dash-dev-branches/releases/tag/v23.0.0-devpr6949.5046964d. The image should be on dockerhub soon. |
|
Guix Automation has began to build this PR tagged as v23.0.0-devpr6949.04b647f5. A new comment will be made when the image is pushed. |
WalkthroughThis pull request updates the Qt dependency from version 5.15.16 to 5.15.18, replacing SHA256 hashes for the main Qt source, qttranslations, and qttools packages. The patch set is modified by removing the clang_18_libpng.patch and adding three new security patches: CVE-2025-4211-qtbase-5.15.patch, CVE-2025-5455-qtbase-5.15.patch, and CVE-2025-30348-qtbase-5.15.patch. The documentation file is updated to reflect the new Qt version reference. Estimated code review effort🎯 4 (Complex) | ⏱️ ~40 minutes
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (6)
💤 Files with no reviewable changes (1)
🧰 Additional context used📓 Path-based instructions (2)doc/**📄 CodeRabbit inference engine (CLAUDE.md)
Files:
depends/**📄 CodeRabbit inference engine (CLAUDE.md)
Files:
🧠 Learnings (2)📓 Common learnings📚 Learning: 2025-08-11T17:16:36.654ZApplied to files:
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (10)
🔇 Additional comments (5)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
Guix Automation has completed; a release should be present here: https://github.com/dashpay/dash-dev-branches/releases/tag/v23.0.0-devpr6949.04b647f5. The image should be on dockerhub soon. |
PastaPastaPasta
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
utACK 04b647f
|
Marked for release in v23.0.1 |
UdjinM6
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
light ACK 04b647f
Motivation
Bitcoin switched to Qt 6.x in bitcoin#30997, upgrading from Qt 5.15.16 to Qt 6.7.3. This transition was enabled by, alongside a series of changes to GUI code, migrating the build system to CMake (see bitcoin#30454). While efforts have been undertaken to bridge the gap between the pre-transition Autotools infrastructure and our infrastructure, migration is complicated by two factors:
This means that the timeline to migration is medium-term at best but in the meanwhile, the latest OSS version of Qt is 5.15.18 (source) and between 5.15.16 and 5.15.19 (the as-of-this-writing, still proprietary release of Qt, source), mitigations for vulnerabilities have been included (source).
While Qt 5.15.19 OSS is not available, critical patches shipped with them are. This pull request updates our Qt depends to the latest available OSS release (v5.15.18) and includes the patches included in the 5.15.19 release.
Additional Information
Guidance on patch application has been taken from the
qt@5Homebrew formula (source).The patches themselves have been sourced from Qt (source) and modified to fit the build's directory structure, it should remain identical otherwise.
*
clang_18_libpng.patchhas been dropped as it is already included in v5.15.18Breaking Changes
None expected.
Checklist