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

fix: C++/CLI mixed mode cannot be used as reference assemblies #297

Merged
merged 4 commits into from
Nov 19, 2024

Conversation

carstencodes
Copy link
Member

dscom crashes, when a C++/CLI Mixed mode assembly is used as a reference.

This is due to the fact, that dotnet cannot load C++/CLI mixed moded assemblies into a collectible AssemblyLoadContext.

Closes #292

For details refer to dotnet/runtime#72237

dscom crashes, when a C++/CLI Mixed mode assembly is used as a reference.

This is due to the fact, that dotnet cannot load C++/CLI mixed moded assemblies into a collectible AssemblyLoadContext.

Closes #292 

For details refer to dotnet/runtime#72237
@electro-logic
Copy link

I like this solution, thanks for this PR. If possible would be fantastic to have an unit test to ensure full compatibility in future with C++/CLI assemblies.

Carsten Igel added 3 commits November 19, 2024 15:06
TLB embedding requires the assemblies to be unloaded
before the embedding process starts.
An AssemblyLoadContext cannot load C++/CLI assemblies, if  it shall be collectible.
dotnet format once more
@carstencodes
Copy link
Member Author

@electro-logic I am currently stuck in a meeting, but come back to this in the course of the day. If you are ok with this, I will take the "BugDemo.zip" you provided us with and add it as an acceptance / end to end test.

@carstencodes carstencodes merged commit 96df7a0 into main Nov 19, 2024
4 checks passed
@carstencodes carstencodes deleted the bugfixes/carstencodes/CppCliNotCollectible branch November 19, 2024 14:22
@electro-logic
Copy link

@carstencodes it's completely fine, thanks

@carstencodes
Copy link
Member Author

#299 is on its way

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] Can't resolve dependencies when using C++/CLI Assemblies
2 participants