Skip to content

Define SystemReflectionMetadataLoadContextVersion property for package references #65614

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

Conversation

MichaelSimons
Copy link
Member

System.Reflection.MetadataLoadContext references in WasmAppBuilder/WasmAppBuilder.csproj were using an incorrect version.props property which breaks source-build. In 6.0.3 SystemReflectionMetadataVersion was revved but not SystemReflectionMetadataLoadContextVersion therefore breaking
the build because a System.Reflection.MetadataLoadContext with the SystemReflectionMetadataVersion does not exist.

@ghost ghost assigned MichaelSimons Feb 19, 2022
@ghost ghost added the area-Build-mono label Feb 19, 2022
@akoeplinger akoeplinger added this to the 6.0.x milestone Feb 21, 2022
@MichaelSimons
Copy link
Member Author

/azp run

@azure-pipelines
Copy link

You have several pipelines (over 10) configured to build pull requests in this repository. Specify which pipelines you would like to run by using /azp run [pipelines] command. You can specify multiple pipelines using a comma separated list.

@MichaelSimons
Copy link
Member Author

@akoeplinger - can you please help with CI. The failures seem unrelated.

…SystemReflectionMetadataLoadContextVersion
@MichaelSimons
Copy link
Member Author

@akoeplinger - Friendly ping, can you please help with CI. The failures seem unrelated.

@akoeplinger
Copy link
Member

The CI failure is unrelated (#61524), but I don't have permissions to merge. @ericstj please merge this one.

@ericstj ericstj merged commit 41e6ff4 into dotnet:release/6.0 Mar 11, 2022
@ghost ghost locked as resolved and limited conversation to collaborators Apr 10, 2022
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.

4 participants