Skip to content
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

Remove old Windows 7 support code #1344

Closed
kennykerr opened this issue Aug 21, 2023 · 2 comments · Fixed by #1348
Closed

Remove old Windows 7 support code #1344

kennykerr opened this issue Aug 21, 2023 · 2 comments · Fixed by #1348

Comments

@kennykerr
Copy link
Collaborator

Probably want to consider dropping Windows 7 support some time as well. That could further simplify this stuff a bit.

Originally posted by @kennykerr in #1342 (comment)

@jonwis
Copy link
Member

jonwis commented Aug 25, 2023

Generally this is righteous. Kind of wish we had better telemetry to see where this is really being used. I'm concerned that if this flows back into the PSDK copy of cppwinrt it'll break people who were somehow relying on it. https://learn.microsoft.com/en-US/lifecycle/faq/extended-security-updates says that there are Win7 variants supported until 2024.

@kennykerr
Copy link
Collaborator Author

Older versions of the C++/WinRT nuget package continue to be the supported way of handling this. This is no different. We don't encourage developers to use the version of C++/WinRT that ships in the Windows SDK since that version is locked to the version that that particular version of the OS was built with which is almost always extremely old (currently Jan 2022). In fact, half the time it doesn't even build with current MSVC or Clang compilers. It exists in the Windows SDK for legacy reasons only.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants