Skip to content

Return the correct OSVersion on OSX like systems.#36029

Merged
eerhardt merged 5 commits into
dotnet:masterfrom
eerhardt:Fix34977
May 11, 2020
Merged

Return the correct OSVersion on OSX like systems.#36029
eerhardt merged 5 commits into
dotnet:masterfrom
eerhardt:Fix34977

Conversation

@eerhardt
Copy link
Copy Markdown
Member

@eerhardt eerhardt commented May 7, 2020

Fix #34977

Here's a link to the underlying API being invoked. I tested this code both on a .NET 5 console app on macOS 10.15.4 and in an iOS simulator running iOS 13.4.1. The method returned the respective version for each. I didn't test tvOS though - I don't have a device.

Comment thread src/libraries/System.Private.CoreLib/src/System/Environment.OSX.cs
Comment thread src/libraries/System.Private.CoreLib/src/System/Environment.OSX.cs Outdated
@marek-safar marek-safar requested a review from rolfbjarne May 8, 2020 08:53
Comment thread src/libraries/System.Private.CoreLib/src/System/Environment.OSX.cs Outdated
Comment thread src/libraries/System.Private.CoreLib/src/System/Environment.OSX.cs Outdated
eerhardt added 3 commits May 8, 2020 13:05
Move interop code to Common\src\Interop.
Specify the full path to libobjc.dylib.
Fix objc_msgSend_stret for ARM64.
Comment thread src/libraries/Common/src/Interop/OSX/Interop.Libraries.cs Outdated
@eerhardt eerhardt merged commit 2d4acbd into dotnet:master May 11, 2020
@eerhardt eerhardt deleted the Fix34977 branch May 11, 2020 14:47
@ghost ghost locked as resolved and limited conversation to collaborators Dec 9, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Change Environment.OSVersion to return actual OS version on macOS

7 participants