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

Clean-up ILLink targets and avoid binplacing #80102

Merged
merged 1 commit into from
Jan 4, 2023

Conversation

ViktorHofer
Copy link
Member

Clean-up illink.targets file, remove outdated logic that doesn't work and isn't used anymore. Avoid binplacing and instead use the P2P protocol to communicate a projects illink settings. Such information is then available in consuming projects like sfx.proj and oob.proj and avoids IO operations (binplacing).

Clean-up illink.targets file, remove outdated logic that doesn't work
and isn't used anymore. Avoid binplacing and instead use the P2P
protocol to communicate a projects illink settings. Such information is
then available in consuming projects like sfx.proj and oob.proj and
avoids IO operations (binplacing).
@ghost
Copy link

ghost commented Jan 2, 2023

Tagging subscribers to this area: @dotnet/area-infrastructure-libraries
See info in area-owners.md if you want to be subscribed.

Issue Details

Clean-up illink.targets file, remove outdated logic that doesn't work and isn't used anymore. Avoid binplacing and instead use the P2P protocol to communicate a projects illink settings. Such information is then available in consuming projects like sfx.proj and oob.proj and avoids IO operations (binplacing).

Author: ViktorHofer
Assignees: ViktorHofer
Labels:

area-Infrastructure-libraries

Milestone: -

@build-analysis build-analysis bot mentioned this pull request Jan 3, 2023
@ViktorHofer ViktorHofer merged commit abc04f2 into dotnet:main Jan 4, 2023
@ViktorHofer ViktorHofer deleted the CleanupILLinkTargets branch January 4, 2023 07:08
@ghost ghost locked as resolved and limited conversation to collaborators Feb 3, 2023
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.

2 participants