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

[release/6.0] Allow nfloat to be in the ObjCRuntime namespace, and make it work for Xamarin.MacCatalyst.dll as well. #60861

Merged
merged 1 commit into from
Nov 11, 2021

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Oct 26, 2021

Backport of #60854 to release/6.0

/cc @lambdageek @rolfbjarne

Customer Impact

Allows applications targeting MacCatalyst to use APIs defined in terms of the nfloat type. Additionally allows the definition of nfloat to be in the ObjCRuntime namespace.

Testing

CI and manual testing

Risk

Low

@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this PR. If you have write-permissions please help me learn by adding exactly one area label.

@lambdageek lambdageek added the Servicing-consider Issue for next servicing release review label Oct 26, 2021
@lambdageek lambdageek added this to the 6.0.1 milestone Oct 26, 2021
@danmoseley
Copy link
Member

@marek-safar we weren't able to represent this at tactics today, could you please do this by email.

@danmoseley
Copy link
Member

fyi @dotnet/jit-contrib a jit test failed here, but with no log captured.

Console log: 'JIT.Methodical.a-dA-D' from job 815d9fa9-2c56-435c-8400-8b4dfb82a69f workitem e007438e-7141-4761-ba97-987e1412c1ad (windows.10.arm64v8.open) executed on machine DDARM64-179

@lambdageek lambdageek modified the milestones: 6.0.1, 6.0.x Nov 11, 2021
@lewing lewing added Servicing-approved Approved for servicing release and removed Servicing-consider Issue for next servicing release review labels Nov 11, 2021
@Anipik Anipik merged commit e8c7c04 into release/6.0 Nov 11, 2021
@jkotas jkotas deleted the backport/pr-60854-to-release/6.0 branch November 13, 2021 22:59
@ghost ghost locked as resolved and limited conversation to collaborators Dec 14, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-Interop-mono os-maccatalyst MacCatalyst OS Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants