Skip to content

Fix broken link in README.md#420

Merged
KirillOsenkov merged 2 commits into
gluck:masterfrom
HebaruSan:patch-1
Apr 24, 2026
Merged

Fix broken link in README.md#420
KirillOsenkov merged 2 commits into
gluck:masterfrom
HebaruSan:patch-1

Conversation

@HebaruSan

Copy link
Copy Markdown
Contributor
  • Update README.md #413's text gave me some false hope that I could merge some .NET 10.0 assemblies until I checked the allowed values of /targetplatform again. Now ".NET assemblies" is replaced by ".NET Framework assemblies."
  • The original link to ILMerge is now broken, so I updated it to the repo that works.

@KirillOsenkov

Copy link
Copy Markdown
Collaborator

what issue did you see merging net10.0 assemblies? I don't know of reasons why it shouldn't work. Perhaps file an issue?

@HebaruSan

HebaruSan commented Apr 23, 2026

Copy link
Copy Markdown
Contributor Author

Huh, it's somewhat encouraging that you expected it to work. The specific error I got was:

ERROR: Loading C:/path/to/file.exe failed: ILRepack does not support merging non-.NET libraries (e.g.: native libraries)

As far as I know, none of the DLLs or EXEs I'm merging are non-.NET libraries, and the equivalent build artifacts work just fine for net481, so I guessed that it was having trouble with versions of .NET post-Framework.

@HebaruSan

Copy link
Copy Markdown
Contributor Author

Well, should I just close this, or do you want to keep the broken link fix?

@HebaruSan

Copy link
Copy Markdown
Contributor Author

Confirmed as user error; I forgot that EXE files are no longer "assemblies" post-Framework.
When I try only repacking DLLs, it works fine, and I can even launch the resulting application.

@HebaruSan HebaruSan changed the title Specify .NET Framework and fix broken link in README.md Fix broken link in README.md Apr 24, 2026

@KirillOsenkov KirillOsenkov left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

👍🏻

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.

2 participants