Skip to content

Tfm clean up#37623

Merged
ericstj merged 3 commits intodotnet:masterfrom
Anipik:tfmCleanUp
Jun 12, 2020
Merged

Tfm clean up#37623
ericstj merged 3 commits intodotnet:masterfrom
Anipik:tfmCleanUp

Conversation

@Anipik
Copy link
Copy Markdown
Contributor

@Anipik Anipik commented Jun 8, 2020

Fixes #34653

  • removes unnecessary cross targeting
  • compiling against system.runtime where possible

@ghost
Copy link
Copy Markdown

ghost commented Jun 8, 2020

Tagging subscribers to this area: @ViktorHofer
Notify danmosemsft if you want to be subscribed.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Can this be changed to only reference public contracts?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

no, because some of the public types conflict with system.private.corelib

@ericstj
Copy link
Copy Markdown
Member

ericstj commented Jun 12, 2020

Only failures are ARM64 timeouts which @safern let me know he's looking into. Will see if I can file an issue on that. Merging now.

@ericstj
Copy link
Copy Markdown
Member

ericstj commented Jun 12, 2020

#37823 for arm64 timeouts

@Anipik Anipik deleted the tfmCleanUp branch August 25, 2020 21:52
@ghost ghost locked as resolved and limited conversation to collaborators Dec 8, 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.

Cleanup TargetFrameworks from project that Reference System.Private.CoreLib

5 participants