-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Certain files should be optional when publishing app as self-contained #27336
Comments
See dotnet/runtime#43716 for an example of customer report on this. |
This sounds reasonable, but we'd need a more concrete proposal from the owners of these files - are there logical groups that would be turned off as a whole, for example? |
Do I understand that createdump.exe will be signed and packaged with apps by default? This could be a significant security concern. |
Does this issue cover the inclusion of unmanaged WPF DLLs in an app that uses Windows Forms but not WPF? |
@tommcdon could you please comment on this?
Probably the better place for this discussions is dotnet/winforms#3723 It's not part of this issue - this issue is about files which are included because some features of the runtime or developer experience make use of them. Mostly there's no way to tell if they are needed by just looking at the code of the app (this is sort of not true for msquic but it depends on the scenario). Inclusion of WPF dlls in winform apps is mainly a problem for trimming which would require two things:
There's one more option - omit WPF dlls (even the managed ones) if |
yes |
I am getting the env: .net7, docker, linux |
For example:
The output directory will contain files like:
createdump.exe
- only useful when trying to create a managed dump of the app's processclrgc.dll
- only useful when wanting to use different GC for the app (otherwise the default GC is part ofcoreclr.dll
)msquic.dll
- only useful when using HTTP3 functionalitymscordaccore*
andmscoredbi*
- only useful for debuggingCurrently there's no way to tell the SDK to exclude some of these. Even with trimming turned on, some of these files are still there (and not used).
We should reconsider the list of files included in self-contained apps by default and if it would make sense to provide options to either opt-in or opt-out for some of them.
/cc @richlander
The text was updated successfully, but these errors were encountered: