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

[linker] Split CoreTypeMapStep into two parts #11474

Merged
merged 1 commit into from
May 7, 2021

Conversation

spouliot
Copy link
Contributor

@spouliot spouliot commented May 6, 2021

The newly extracted RegistrarRemovalTrackingStep can be used inside
dotnet-linker to remove the dynamic registrar (if not required by some
other code).

Size MySingleView.app
4,579,969 legacy
11,182,125 dotnet before
10,760,493 dotnet after

The newly extracted `RegistrarRemovalTrackingStep` can be used inside
`dotnet-linker` to remove the dynamic registrar (if not required by some
other code).
@spouliot spouliot requested a review from rolfbjarne as a code owner May 6, 2021 23:22
@spouliot spouliot added the not-notes-worthy Ignore for release notes label May 6, 2021
Copy link
Member

@dalexsoto dalexsoto left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

shave away those bits! 🪒

@vs-mobiletools-engineering-service2
Copy link
Collaborator

❌ [PR Build] Tests failed on Build ❌

Tests failed on Build.

API diff

✅ API Diff from stable

View API diff

API & Generator diff

API Diff (from PR only) (no change)
Generator Diff (no change)

Test results

1 tests failed, 106 tests passed.

Failed tests

  • xcframework-test/Mac Catalyst/Debug: TimedOut (Execution timed out after 1200 seconds.
    No test log file was produced)

Pipeline on Agent XAMBOT-1094.BigSur
Merge c132aa0 into 5fba609

@spouliot
Copy link
Contributor Author

spouliot commented May 7, 2021

xcframework-test/Mac Catalyst/Debug: TimedOut (Execution timed out after 1200 seconds.
No test log file was produced)

unrelated failure, known issue tracking in https://github.com/xamarin/maccore/issues/2414

@spouliot spouliot merged commit ccd769b into xamarin:main May 7, 2021
@spouliot spouliot deleted the split-coretypemapstep branch May 7, 2021 12:20
@spouliot spouliot mentioned this pull request Jun 4, 2021
6 tasks
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.

4 participants