-
Notifications
You must be signed in to change notification settings - Fork 4.7k
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
Delete Microsoft.Windows.Compatibility from runtime #84013
Delete Microsoft.Windows.Compatibility from runtime #84013
Conversation
dotnet/winforms#8909 moves the M.W.C package into windowsdesktop. Deleting the source from runtime and the package versions that aren't used anymore.
Tagging subscribers to this area: @dotnet/area-infrastructure-libraries Issue Detailsdotnet/winforms#8909 moves the M.W.C package into windowsdesktop. Deleting the source from runtime and the package versions that aren't used anymore.
|
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.
Assuming the CI has unrelated failures, LGTM.
<SystemRuntimeInteropServicesRuntimeInformationVersion>4.3.0</SystemRuntimeInteropServicesRuntimeInformationVersion> | ||
<SystemSecurityAccessControlVersion>6.0.0</SystemSecurityAccessControlVersion> | ||
<SystemSecurityCryptographyAlgorithmsVersion>4.3.1</SystemSecurityCryptographyAlgorithmsVersion> | ||
<SystemSecurityCryptographyCngVersion>5.0.0</SystemSecurityCryptographyCngVersion> | ||
<SystemSecurityCryptographyOpenSslVersion>5.0.0</SystemSecurityCryptographyOpenSslVersion> | ||
<SystemSecurityPrincipalWindowsVersion>5.0.0</SystemSecurityPrincipalWindowsVersion> | ||
<SystemSecurityPermissionsVersion>7.0.0</SystemSecurityPermissionsVersion> | ||
<SystemServiceModelPrimitivesVersion>4.9.0</SystemServiceModelPrimitivesVersion> |
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.
Are these the packages you mention in the PR description?
and the package versions that aren't used anymore.
I am missing context. Are these related to M.W.C? Or just removing them as an unrelated improvement?
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.
Those versions were referenced only by M.W.C. Now that M.W.C doesn't exist in runtime anymore, delete them as they are dead code.
Closes #77557
Blocked by dotnet/windowsdesktop#3516
dotnet/windowsdesktop#3516 moves the M.W.C package into windowsdesktop. Deleting the source from runtime and the package versions that aren't used anymore.