What the developer wrote
Authored by fanquake
depends: drop unused qt patch
This hasn't been applied since 5e794e62024eef612e1fbb71c76ea54d17435c14,
but the patch was never removed.
What changed, and why it matters
This commit simply deletes an old, unused patch file from the build system. The patch was no longer being applied to Qt, so removing it has no effect on the compiled Bitcoin Core software and does not change any security behavior.
No action needed; this is a benign cleanup commit.
Security signals we found
No strong security signals were identified.
Evidence from the diff
The commit removes depends/patches/qt/memory_resource.patch, a Qt build patch that had not been applied since commit 5e794e6. The patch itself adjusted a feature-test macro in qduplicatetracker_p.h from QT_HAS_INCLUDE(
Changed components
depends/patches/qt/memory_resource.patchInspect captured patch +0 / −16
diff --git a/depends/patches/qt/memory_resource.patch b/depends/patches/qt/memory_resource.patch
deleted file mode 100644
index 14e25121..00000000
--- a/depends/patches/qt/memory_resource.patch
+++ /dev/null
@@ -1,16 +0,0 @@
-Fix unusable memory_resource on macos
-
-See https://bugreports.qt.io/browse/QTBUG-117484
-and https://bugreports.qt.io/browse/QTBUG-114316
-
---- a/qtbase/src/corelib/tools/qduplicatetracker_p.h
-+++ b/qtbase/src/corelib/tools/qduplicatetracker_p.h
-@@ -52,7 +52,7 @@
-
- #include <qglobal.h>
-
--#if QT_HAS_INCLUDE(<memory_resource>) && __cplusplus > 201402L
-+#ifdef __cpp_lib_memory_resource
- # include <unordered_set>
- # include <memory_resource>
- #else
Why this scored 15/100
Community notes
Notes can correct, qualify, or add evidence to the AI analysis. Every note shown here has been validated by a human moderator.
The AI analysis stands alone for now. Submit a note if you can add evidence or important context.