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

[mono] Remove looking at .exe in assembly loader #107908

Closed
wants to merge 1 commit into from

Conversation

akoeplinger
Copy link
Member

Modern .NET doesn't use .exe for managed assemblies.
Contributes to #107883

Modern .NET doesn't use .exe for managed assemblies.
Copy link
Contributor

Tagging subscribers to this area:
See info in area-owners.md if you want to be subscribed.

@akoeplinger
Copy link
Member Author

Hmm looks like we have a couple test failures, because of this code which changes the test assembly extension to .exe:

<!-- To be consistent between net472 and others we override the SDK behavior which defaults to .dll -->
<TargetExt>.exe</TargetExt>

This would need a bit more work so closing for now.

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

Successfully merging this pull request may close these issues.

2 participants