Conversation
Kodiai Review SummaryWhat ChangedBumped dlfcn-win32 dependency from v1.2.0 to v1.4.2, adding CMake 4.0 and Windows ARM64 support, and switching to a forked version for UWP to avoid Azure patch.exe crashes. Reviewed: core logic, docs Strengths
ObservationsImpact[MAJOR] depends/windows/dlfcn-win32/0001-dlopen_with_widechar.patch (22): Missing NULL check after malloc [MEDIUM] depends/windowsstore/dlfcn-win32/dlfcn-win32.txt (1): Using forked repository introduces supply chain risk Suggestions
Verdict🔴 Address before merging -- 1 blocking issue(s) found (CRITICAL/MAJOR) Review Details
|
5966679 to
aca0c4f
Compare
Description
As title says, I bumped dlfcn-win32 to v1.4.2.
Notable changes:
dlfcn-win32 fork
I rebased the windows UWP patch, but it crashes patch.exe on Azure, so I forked the dlfcn-win32 repo and tagged v1.4.2 with Alwin's commit.
Error on azure was:
Then in the depends file I set the repo to my fork + the commit:
How has this been tested?
Compile-tested on Windows 10 x64. Runtime testing in progress.
Other add-ons
The following add-ons use dlfcn-win32 and should be updated: