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

WindowsDesktop and ASP.NET are not considered linkable frameworks #49252

Closed
agocke opened this issue Mar 5, 2021 · 6 comments
Closed

WindowsDesktop and ASP.NET are not considered linkable frameworks #49252

agocke opened this issue Mar 5, 2021 · 6 comments
Labels
area-Meta linkable-framework Issues associated with delivering a linker friendly framework
Milestone

Comments

@agocke
Copy link
Member

agocke commented Mar 5, 2021

When trimming any WinForms, WPF, or ASP.NET, the whole framework is not linked because it is considered user code.

This may not be relevant if we do not support trimming these frameworks, but otherwise we should mark them as linkable as soon as we have fixed preliminary warnings.

@dotnet-issue-labeler dotnet-issue-labeler bot added the untriaged New issue has not been triaged by the area owner label Mar 5, 2021
@dotnet-issue-labeler
Copy link

I couldn't figure out the best area label to add to this issue. If you have write-permissions please help me learn by adding exactly one area label.

@agocke agocke added this to the 6.0.0 milestone Mar 5, 2021
@agocke agocke added area-Meta linkable-framework Issues associated with delivering a linker friendly framework labels Mar 5, 2021
@ghost
Copy link

ghost commented Mar 5, 2021

Tagging subscribers to 'linkable-framework': @eerhardt, @vitek-karas, @LakshanF, @tannergooding, @sbomer
See info in area-owners.md if you want to be subscribed.

Issue Details

When trimming any WinForms, WPF, or ASP.NET, the whole framework is not linked because it is considered user code.

This may not be relevant if we do not support trimming these frameworks, but otherwise we should mark them as linkable as soon as we have fixed preliminary warnings.

Author: agocke
Assignees: -
Labels:

area-Meta, linkable-framework, untriaged

Milestone: 6.0.0

@agocke agocke removed the untriaged New issue has not been triaged by the area owner label Mar 5, 2021
@eerhardt
Copy link
Member

eerhardt commented Mar 6, 2021

We have a feature now that any library can add [assembly:AssemblyMetadata("IsTrimmable", "True")], and when TrimMode=link that assembly will be trimmed.

See #48428
https://github.com/mono/linker/blob/main/docs/design/trimmed-assemblies.md#assemblymetadataistrimmable-true.

I think this issue can be closed now in dotnet/runtime. It is possible for those libraries to "self-serve" here. No work is needed in dotnet/runtime.

@marek-safar
Copy link
Contributor

Yep, there is not much left to do in runtime. It'd be better to open an issue in WinForms, WPF repos with links on how to make their SDK trimmable (that's what we did for other repos).

@eerhardt
Copy link
Member

eerhardt commented Mar 9, 2021

Closing as per above discussion.

@eerhardt eerhardt closed this as completed Mar 9, 2021
@agocke
Copy link
Member Author

agocke commented Mar 9, 2021

Thanks, the guidance above looks good.

@ghost ghost locked as resolved and limited conversation to collaborators Apr 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-Meta linkable-framework Issues associated with delivering a linker friendly framework
Projects
No open projects
Development

No branches or pull requests

3 participants