-
Couldn't load subscription status.
- Fork 1.4k
Expose static methods from System.OperatingSystem #8935
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
Expose static methods from System.OperatingSystem #8935
Conversation
Co-authored-by: Alexander Köplinger <[email protected]>
Co-authored-by: Alexander Köplinger <[email protected]>
Co-authored-by: Alexander Köplinger <[email protected]>
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.
Looks good to me apart from one comment. 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.
Thanks!
This reverts commit 759ae6a.
Fixes #5982
Context
Expose methods added in .Net 5.0. Create alternate stand-in for net471.
Changes Made
Microsoft.Build.Framework.OperatingSystemas stand-in substitute for net471. The class assumes that only Windows is supported.Microsoft.Build.Framework.OperatingSystem.System.OperatingSystemor fromMicrosoft.Build.Framework.OperatingSystem.IsOSPlatformproperty functionTesting
Tested on windows 11 and macOS 12. tested by running full suite of unit test.
Notes