Skip to content

Fix loading of refonly assemblies for "classic" apps - #7094

Merged
grendello merged 1 commit into
dotnet:mainfrom
grendello:fix-refonly-assemblies
Jun 14, 2022
Merged

grendello merged 1 commit into
dotnet:mainfrom
grendello:fix-refonly-assemblies

Conversation

@grendello

Copy link
Copy Markdown
Contributor

Fixes: #7091
Context: e1af958

e1af958 refactored a handful of MonoImage loading methods and
during that process, I mistakenly made reference-only assemblies
to be loaded in a "full" way which broke a number of BCL tests.

Fix the issue by making sure reference-only assemblies are really
loaded as such.

Fixes: dotnet#7091
Context: dotnet@e1af958

e1af958 refactored a handful of `MonoImage` loading methods and
during that process, I mistakenly made reference-only assemblies
to be loaded in a "full" way which broke a number of BCL tests.

Fix the issue by making sure reference-only assemblies are really
loaded as such.
@grendello
grendello requested a review from jonpryor as a code owner June 14, 2022 13:20
@grendello
grendello requested a review from pjcollins June 14, 2022 13:20
@pjcollins

Copy link
Copy Markdown
Member

We currently have issues with test result publishing in CI, but looking at the job the test step is now passing so I think this should be ready to merge:
image

@grendello
grendello merged commit 8070679 into dotnet:main Jun 14, 2022
@grendello
grendello deleted the fix-refonly-assemblies branch June 14, 2022 17:25
@github-actions github-actions Bot locked and limited conversation to collaborators Jan 24, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

No longer seeing InvalidOperationExceptions with certain Assembly.ReflectionOnlyLoad usage

3 participants