Skip to content

[runtime] Redirect to our objc_msgSend wrapper functions when needed for .NET code.#10932

Merged
rolfbjarne merged 5 commits intodotnet:mainfrom
rolfbjarne:coreclr-objc-msgsend
Mar 24, 2021
Merged

[runtime] Redirect to our objc_msgSend wrapper functions when needed for .NET code.#10932
rolfbjarne merged 5 commits intodotnet:mainfrom
rolfbjarne:coreclr-objc-msgsend

Conversation

@rolfbjarne
Copy link
Member

This makes the mono_dllmap_insert function unnecessary for .NET, so remove it.

Ref: dotnet/runtime#48110
Ref: dotnet/runtime#43204
Ref: #10504

…for .NET code.

This makes the mono_dllmap_insert function unnecessary for .NET, so remove it.

Ref: dotnet/runtime#48110
Ref: dotnet/runtime#43204
Ref: dotnet#10504
… apps due to a missing feature in the runtime. (dotnet#10123)"

This reverts commit 75979ca.
@rolfbjarne rolfbjarne requested a review from spouliot as a code owner March 22, 2021 17:54
@rolfbjarne rolfbjarne added the not-notes-worthy Ignore for release notes label Mar 22, 2021
@spouliot
Copy link
Contributor

There's mention of mono support in dotnet/runtime#48110 (comment)

@CoffeeFlux is that available in 2020-02 by any chance ?

@CoffeeFlux
Copy link

@spouliot sadly no, all the CoreCLR hosting API stuff (including monovm_initialize) is net6-only.

@vs-mobiletools-engineering-service2
Copy link
Collaborator

❌ Tests failed on Build ❌

Tests failed on Build.

API diff

✅ API Diff from stable

View API diff

Test results

4 tests failed, 178 tests passed.

Failed tests

  • monotouch-test/iOS Unified 64-bits - simulator/Debug [dotnet]: Crashed
  • monotouch-test/iOS Unified 64-bits - simulator/Debug (LinkSdk) [dotnet]: Crashed
  • monotouch-test/iOS Unified 64-bits - simulator/Debug (static registrar) [dotnet]: Crashed
  • monotouch-test/iOS Unified 64-bits - simulator/Debug (all optimizations) [dotnet]: Crashed

Pipeline on Agent XAMBOT-1099

@vs-mobiletools-engineering-service2
Copy link
Collaborator

❌ Tests failed on Build ❌

Tests failed on Build.

API diff

✅ API Diff from stable

View API diff

Test results

1 tests failed, 181 tests passed.

Failed tests

  • introspection/watchOS 32-bits - simulator/Debug (watchOS 3.2): LaunchFailure Known issue: HE0038)

Pipeline on Agent XAMBOT-1097

@rolfbjarne rolfbjarne merged commit 9153ede into dotnet:main Mar 24, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

not-notes-worthy Ignore for release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants