-
Notifications
You must be signed in to change notification settings - Fork 1.2k
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
Simplify dwrite.dll loading process #6538
Conversation
All supported OS always has `dwrite.dll`.
} | ||
} | ||
|
||
HMODULE hDWriteLibrary = LoadLibraryEx(L"dwrite.dll", nullptr, LOAD_LIBRARY_SEARCH_SYSTEM32); |
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.
Windows 7, Windows Server 2008 R2, Windows Vista and Windows Server 2008: This value requires KB2533623 to be installed. See https://docs.microsoft.com/en-us/windows/win32/api/libloaderapi/nf-libloaderapi-loadlibraryexa
Thank you.
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.
Argh! I miss that. But only Windows 7 SP1 is possible applicable since other OS does not supported by .NET 7
And even that seems to be KB2533623 required for .NET see dotnet/docs#20459
But I hardly can understand is ESU has this patch or not.
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.
Vista and 2008 Server are not in the list of supported OS (at least for .NET 6) (research took too long, at time of answering kant2002 answer was not there)
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.
That should be slated for .NET 8 then. That way Win 7 would be out of support at release date (Nov 2023) and not important anymore.
@kant2002 thanks for your contribution. |
All supported OS always has
dwrite.dll
.Microsoft Reviewers: Open in CodeFlow