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-preview7] Add explicit dependency on System.Reflection.Metadata to RuntimeConfigParser task #56216

Merged

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jul 23, 2021

Backport of #56152 to release/6.0-preview7

/cc @steveisok @directhex

Customer Impact

The RuntimeConfigParser task took a dependency on System.Reflection.Metadata, but did not include all of the assemblies when targeting full framework. Without this change, building android and iOS projects in VS would fail.

Testing

We updated the android & ios workloads to contain these changes and can build an android project using full framework MSBuild successfully.

Risk

Very 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.

@steveisok steveisok added the Servicing-approved Approved for servicing release label Jul 23, 2021
@steveisok
Copy link
Member

Part of the VS installer work, which has been pre-approved.

@mmitche mmitche merged commit c8ec393 into release/6.0-preview7 Jul 23, 2021
@mmitche mmitche deleted the backport/pr-56152-to-release/6.0-preview7 branch July 23, 2021 22:22
@ghost ghost locked as resolved and limited conversation to collaborators Aug 22, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Servicing-approved Approved for servicing release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants