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

Set the ReferenceAssembly metadata on the manually added ReferencePath items #109433

Merged
merged 1 commit into from
Nov 1, 2024

Conversation

jkoritzinsky
Copy link
Member

@jkoritzinsky jkoritzinsky commented Oct 31, 2024

Unblocks using runtime with a newer arcade that actually respects Roslyn ref assemblies.

Needed to unblock the dotnet/sdk VMR build.

…h items

Unblocks using runtime with a newer arcade that actually respects Roslyn ref assemblies.
@jkoritzinsky jkoritzinsky added blocking Marks issues that we want to fast track in order to unblock other important work area-Infrastructure and removed area-Setup labels Oct 31, 2024
@jkoritzinsky jkoritzinsky added this to the 10.0.0 milestone Oct 31, 2024
Copy link
Contributor

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

@@ -17,6 +17,7 @@
<Target Name="AddFrameworkFilesToPackage" DependsOnTargets="ResolveLibrariesFromLocalBuild" BeforeTargets="GetFilesToPackage">
<ItemGroup>
<ReferencePath Include="@(LibrariesRefAssemblies)" />
<ReferencePath ReferenceAssembly="%(Identity)" />
Copy link
Member

Choose a reason for hiding this comment

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

Where's the metadata used?

Copy link
Member Author

Choose a reason for hiding this comment

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

This is used in the SharedFramework SDK now (as of the arcade update PR that's currently in flight) to select the (usually Roslyn-produced) ref assembly instead of the implementation assembly when building a ref pack.

Copy link
Member Author

Choose a reason for hiding this comment

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

This was changed in dotnet/arcade#15209

@jkoritzinsky jkoritzinsky merged commit 6406b57 into dotnet:main Nov 1, 2024
78 of 79 checks passed
@jkoritzinsky jkoritzinsky deleted the refpack-metadata branch November 1, 2024 01:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-Infrastructure blocking Marks issues that we want to fast track in order to unblock other important work
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants